commit 35aa9a889fda41d75a35633faba1b081c70ce86c
author Patrick Williams <patrick@stwcx.xyz> Tue Apr 05 15:55:30 2022 -0500
committer Patrick Williams <patrick@stwcx.xyz> Thu Apr 07 22:02:30 2022 -0500
tree ee651cfc7c16b3a347036b55b85a358befe74946
parent 703de25c2ac0e215f1ac692ac2ebfd9fe93af380

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

activation.hpp

diff --git a/activation.hpp b/activation.hpp
index 90b32b7..7425a54 100644
--- a/activation.hpp
+++ b/activation.hpp
@@ -189,7 +189,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,