commit 9c887d14c4ad36401fadc35cfdffb6089d4ee22a
author Patrick Williams <patrick@stwcx.xyz> Tue Apr 05 21:39:40 2022 -0500
committer Adriana Kobylak <anoo@linux.ibm.com> Tue Apr 12 17:51:12 2022 +0000
tree b6eede6a8b8e89e7d98a6e5d5f5045f76c066d9c
parent 2b78eb0ef42c49fde43684f542832576db07253b

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` or `action::emit_interface_added`.

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: Id3689b24f8719bafa9e5a67960848edff4ddf8c5

activation.hpp

diff --git a/activation.hpp b/activation.hpp
index 0622545..cc2df97 100644
--- a/activation.hpp
+++ b/activation.hpp
@@ -158,7 +158,8 @@
                sdbusplus::xyz::openbmc_project::Software::server::Activation::
                    Activations activationStatus,
                AssociationList& assocs) :
-        ActivationInherit(bus, path.c_str(), true),
+        ActivationInherit(bus, path.c_str(),
+                          ActivationInherit::action::defer_emit),
         bus(bus), path(path), parent(parent), versionId(versionId),
         systemdSignals(
             bus,