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

host_state_manager.hpp

diff --git a/host_state_manager.hpp b/host_state_manager.hpp
index 1e9bb10..6b9d687 100644
--- a/host_state_manager.hpp
+++ b/host_state_manager.hpp
@@ -54,7 +54,7 @@
      * @param[in] id        - The Host id
      */
     Host(sdbusplus::bus::bus& bus, const char* objPath, size_t id) :
-        HostInherit(bus, objPath, true), bus(bus),
+        HostInherit(bus, objPath, HostInherit::action::defer_emit), bus(bus),
         systemdSignalJobRemoved(
             bus,
             sdbusRule::type::signal() + sdbusRule::member("JobRemoved") +