sdbusplus: object: don't use 'bool' argument constructor
`sdbusplus::server::object_t` has long had an enum-based parameter for
signal action, but maintained a backwards compatible boolean mapping.
It is time to remove this boolean to make it more observable which
actions are being used in applications. Map all `true` occurrences to
`action::defer_emit` or `action::emit_no_signals` as appropriate.
- Value: defer_emit (primary object)
- Others: emit_no_signals (secondary objects)
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I892a1c953231123aa5407b388ff9133882d80feb
diff --git a/fan_speed.hpp b/fan_speed.hpp
index 4b79b2a..f1a3fdb 100644
--- a/fan_speed.hpp
+++ b/fan_speed.hpp
@@ -33,7 +33,9 @@
const std::string& devPath, const std::string& id,
sdbusplus::bus::bus& bus, const char* objPath, bool defer,
uint64_t target) :
- FanSpeedObject(bus, objPath, defer),
+ FanSpeedObject(bus, objPath,
+ defer ? FanSpeedObject::action::emit_no_signals
+ : FanSpeedObject::action::emit_object_added),
_id(id), _ioAccess(std::move(io)), _devPath(devPath)
{
FanSpeedObject::target(target);