commit 76070747a514ca68c1a37e1690723c9965a4abf9
author Patrick Williams <patrick@stwcx.xyz> Tue Apr 05 15:20:12 2022 -0500
committer Patrick Williams <patrick@stwcx.xyz> Thu Apr 07 14:22:41 2022 -0500
tree 6f82c58e4f080fcfd4a0a9b15c314133369f140e
parent 003f6fa28070902759313557a1b57831db13e651

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: I5cfac1360e8c769889b33bef98cfd44778018a44

chassis_state_manager.hpp

diff --git a/chassis_state_manager.hpp b/chassis_state_manager.hpp
index 1dc7716..3bd44cd 100644
--- a/chassis_state_manager.hpp
+++ b/chassis_state_manager.hpp
@@ -47,7 +47,8 @@
      * @param[in] id        - Chassis id
      */
     Chassis(sdbusplus::bus::bus& bus, const char* objPath, size_t id) :
-        ChassisInherit(bus, objPath, true), bus(bus),
+        ChassisInherit(bus, objPath, ChassisInherit::action::defer_emit),
+        bus(bus),
         systemdSignals(
             bus,
             sdbusRule::type::signal() + sdbusRule::member("JobRemoved") +