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: I12bac733597177ec85336a204b081779fe6ce19e
diff --git a/snmp_conf_manager.cpp b/snmp_conf_manager.cpp
index ed73daf..d77f8b7 100644
--- a/snmp_conf_manager.cpp
+++ b/snmp_conf_manager.cpp
@@ -25,7 +25,8 @@
using Argument = xyz::openbmc_project::Common::InvalidArgument;
ConfManager::ConfManager(sdbusplus::bus::bus& bus, const char* objPath) :
- details::CreateIface(bus, objPath, true),
+ details::CreateIface(bus, objPath,
+ details::CreateIface::action::defer_emit),
dbusPersistentLocation(SNMP_CONF_PERSIST_PATH), bus(bus),
objectPath(objPath)
{}