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

bmc_state_manager.hpp

diff --git a/bmc_state_manager.hpp b/bmc_state_manager.hpp
index ee97070..239ad10 100644
--- a/bmc_state_manager.hpp
+++ b/bmc_state_manager.hpp
@@ -34,7 +34,7 @@
      * @param[in] objPath   - The Dbus object path
      */
     BMC(sdbusplus::bus::bus& bus, const char* objPath) :
-        BMCInherit(bus, objPath, true), bus(bus),
+        BMCInherit(bus, objPath, BMCInherit::action::defer_emit), bus(bus),
         stateSignal(std::make_unique<decltype(stateSignal)::element_type>(
             bus,
             sdbusRule::type::signal() + sdbusRule::member("JobRemoved") +