dbus2 / zbus-old

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

dbus_proxy: consider switching 'assume_defaults' default value to 'false' #298

Open zeenix opened 1 year ago

zeenix commented 1 year ago

In GitLab by @lucab on Nov 8, 2022, 17:24

This is a followup from https://gitlab.freedesktop.org/dbus/zbus/-/issues/284#note_1624111.

The patch at https://gitlab.freedesktop.org/dbus/zbus/-/merge_requests/587 is introducing a new assume_defaults attribute for the dbus_proxy macro. For compatibility reasons, it is initially defaulting to true (i.e. the implicit behavior beforehand) and producing a compiler warning when omitted (and not default destination/path are provided).

On the next semver break, it would be good to consider switching the default to assume_defaults = false and adjust the compiler warning.