sdbusplus - const/readonly flags
anoo at linux.ibm.com
Wed Aug 26 06:52:42 AEST 2020
On 2020-08-25 10:00, Patrick Williams wrote:
> * I expect some of the 'const' flags in phosphor-dbus-interfaces are
> wrong and should be changed to 'readonly'. I have collected a
> of them in a gist. If you really intend to mean "this property
> will never change during the life of an object" continue to use
> 'const', but if you mean "this property shall not be changed by
> clients", use 'readonly' (and probably also add 'emits_change').
There are also some properties in phosphor-dbus-interfaces with
descriptions that say "read-only property", and the implementation is
enforcing it. It may good for the owners to revisit those as well and
see if a 'read-only' flag is applicable.
> 2. https://github.com/openbmc/sdbusplus/issues/48
More information about the openbmc