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_mapper_entry.cpp b/phosphor-ldap-config/ldap_mapper_entry.cpp
index eb4a4f2..b9442e3 100644
--- a/phosphor-ldap-config/ldap_mapper_entry.cpp
+++ b/phosphor-ldap-config/ldap_mapper_entry.cpp
@@ -27,7 +27,7 @@
const char* filePath,
const std::string& groupName,
const std::string& privilege, Config& parent) :
- Interfaces(bus, path, true),
+ Interfaces(bus, path, Interfaces::action::defer_emit),
id(std::stol(std::filesystem::path(path).filename())), manager(parent),
persistPath(filePath)
{
@@ -38,7 +38,7 @@
LDAPMapperEntry::LDAPMapperEntry(sdbusplus::bus::bus& bus, const char* path,
const char* filePath, Config& parent) :
- Interfaces(bus, path, true),
+ Interfaces(bus, path, Interfaces::action::defer_emit),
id(std::stol(std::filesystem::path(path).filename())), manager(parent),
persistPath(filePath)
{}