commit 6ef6b25e44e07e1af74f2578d30331922c7879ac
author Patrick Williams <patrick@stwcx.xyz> Wed Mar 30 14:28:27 2022 -0500
committer Patrick Williams <patrick@stwcx.xyz> Sat Apr 02 11:47:06 2022 +0000
tree d620956d169e98d293e18da065b35b463a9e45fc
parent aa85a0729ce26eb3a9b759fb87fb96d5caa3b7a3

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`.

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

phosphor-rsyslog-config/server-conf.hpp

diff --git a/phosphor-rsyslog-config/server-conf.hpp b/phosphor-rsyslog-config/server-conf.hpp
index 979f845..5b3c5a1 100644
--- a/phosphor-rsyslog-config/server-conf.hpp
+++ b/phosphor-rsyslog-config/server-conf.hpp
@@ -40,7 +40,7 @@
      */
     Server(sdbusplus::bus::bus& bus, const std::string& path,
            const char* filePath) :
-        Iface(bus, path.c_str(), true),
+        Iface(bus, path.c_str(), Iface::action::defer_emit),
         configFilePath(filePath),
         hostnameChange(
             bus,