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

hypervisor_state_manager.hpp

diff --git a/hypervisor_state_manager.hpp b/hypervisor_state_manager.hpp
index 9bb3cb2..929c1aa 100644
--- a/hypervisor_state_manager.hpp
+++ b/hypervisor_state_manager.hpp
@@ -41,7 +41,9 @@
      * @param[in] objPath   - The Dbus object path
      */
     Hypervisor(sdbusplus::bus::bus& bus, const char* objPath) :
-        HypervisorInherit(bus, objPath, false), bus(bus),
+        HypervisorInherit(bus, objPath,
+                          HypervisorInherit::action::emit_object_added),
+        bus(bus),
         bootProgressChangeSignal(
             bus,
             sdbusRule::propertiesChanged(