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`.
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I94aa9a4451cebc772d3259323ff1b679e53f90bb
diff --git a/pid/zone.hpp b/pid/zone.hpp
index 37b49ea..01dc62e 100644
--- a/pid/zone.hpp
+++ b/pid/zone.hpp
@@ -38,7 +38,9 @@
DbusPidZone(int64_t zone, double minThermalOutput, double failSafePercent,
const SensorManager& mgr, sdbusplus::bus::bus& bus,
const char* objPath, bool defer) :
- ModeObject(bus, objPath, defer),
+ ModeObject(bus, objPath,
+ defer ? ModeObject::action::defer_emit
+ : ModeObject::action::emit_object_added),
_zoneId(zone), _maximumSetPoint(),
_minThermalOutputSetPt(minThermalOutput),
_failSafePercent(failSafePercent), _mgr(mgr)
diff --git a/sensors/host.hpp b/sensors/host.hpp
index 5c70526..bf935b7 100644
--- a/sensors/host.hpp
+++ b/sensors/host.hpp
@@ -48,7 +48,9 @@
HostSensor(const std::string& name, int64_t timeout,
sdbusplus::bus::bus& bus, const char* objPath, bool defer) :
Sensor(name, timeout),
- ValueObject(bus, objPath, defer)
+ ValueObject(bus, objPath,
+ defer ? ValueObject::action::defer_emit
+ : ValueObject::action::emit_object_added)
{}
ValueType value(ValueType value) override;