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: Ia42c547e5f4e7644315dad6f7b13476cd8db4826
diff --git a/src/event_entry.hpp b/src/event_entry.hpp
index 28a6338..1f99d81 100644
--- a/src/event_entry.hpp
+++ b/src/event_entry.hpp
@@ -41,7 +41,8 @@
*/
Entry(const std::string& path, uint64_t eventTimestamp, std::string&& msg,
std::vector<std::string>&& metaData) :
- EntryIface(SDBusPlus::getBus(), path.c_str(), true),
+ EntryIface(SDBusPlus::getBus(), path.c_str(),
+ EntryIface::action::defer_emit),
objectPath(path)
{
timestamp(eventTimestamp);
@@ -57,7 +58,9 @@
* @param[in] timestamp - timestamp when the event created.
*/
Entry(const std::string& path, uint64_t eventTimestamp) :
- EntryIface(SDBusPlus::getBus(), path.c_str(), true), objectPath(path)
+ EntryIface(SDBusPlus::getBus(), path.c_str(),
+ EntryIface::action::defer_emit),
+ objectPath(path)
{
timestamp(eventTimestamp);
}