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: I18604dd269289a35705f52455980798c069a756f
diff --git a/softoff/softoff.hpp b/softoff/softoff.hpp
index 1c9341b..bf3fbdd 100644
--- a/softoff/softoff.hpp
+++ b/softoff/softoff.hpp
@@ -18,11 +18,15 @@
namespace sdbusRule = sdbusplus::bus::match::rules;
+namespace
+{
+using SoftPowerOffInherit = sdbusplus::server::object_t<Base::SoftPowerOff>;
+}
+
/** @class SoftPowerOff
* @brief Responsible for coordinating Host SoftPowerOff operation
*/
-class SoftPowerOff
- : public sdbusplus::server::object::object<Base::SoftPowerOff>
+class SoftPowerOff : public SoftPowerOffInherit
{
public:
/** @brief Constructs SoftPowerOff object.
@@ -33,8 +37,8 @@
*/
SoftPowerOff(sdbusplus::bus::bus& bus, sd_event* event,
const char* objPath) :
- sdbusplus::server::object::object<Base::SoftPowerOff>(bus, objPath,
- false),
+ SoftPowerOffInherit(bus, objPath,
+ SoftPowerOffInherit::action::defer_emit),
bus(bus), timer(event),
hostControlSignal(
bus,