search

dbus2 / zbus-old

Rust D-Bus crate.
https://gitlab.freedesktop.org/dbus/zbus
Other
49 stars 13 forks source link

zm: Mention Value/OwnedValue conversion requirement for properties #328

Closed zeenix closed 1 year ago

zeenix commented 1 year ago

In GitLab by @sre on Feb 18, 2023, 23:21

Hi,

Currently neither https://docs.rs/zbus/latest/zbus/attr.dbus_interface.html nor https://dbus.pages.freedesktop.org/zbus/server.html mention that properties need Value/OwnedValue conversion while normal methods work without that. Since it is IMHO unexpected that property getter/setters handle their arguments differently than normal methods it should be documented.

zeenix commented 1 year ago

!666 adds the mention in the client section because that's where the properties are first introduced. I didn't want to duplicate it in the server section too but feel free to provide a PR for that if you think that'd be good.

zeenix commented 1 year ago

mentioned in commit 73fc41480f4d203f11965bdcb7e4411bb1fd43fb