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: I6de31883b9f6a9a3115ae232575dffbdf652e0bd
diff --git a/phosphor-ldap-config/ldap_config_mgr.hpp b/phosphor-ldap-config/ldap_config_mgr.hpp
index 9ba69d4..470e7f9 100644
--- a/phosphor-ldap-config/ldap_config_mgr.hpp
+++ b/phosphor-ldap-config/ldap_config_mgr.hpp
@@ -56,7 +56,7 @@
ConfigMgr(sdbusplus::bus::bus& bus, const char* path, const char* filePath,
const char* dbusPersistentPath, const char* caCertFile,
const char* certFile) :
- CreateIface(bus, path, true),
+ CreateIface(bus, path, CreateIface::action::defer_emit),
dbusPersistentPath(dbusPersistentPath), configFilePath(filePath),
tlsCacertFile(caCertFile), tlsCertFile(certFile), bus(bus)
{}