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` or `action::emit_no_signals` as appropriate.
- DumpEntry: emit_no_signals (secondary object)
- {bmc,resource,system}::DumpEntry: defer_emit (primary object)
- Manager: defer_emit (primary object)
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I91990031384230550903a46332adc04d691a3cb9
diff --git a/dump_entry.hpp b/dump_entry.hpp
index cbeb012..7a8bba3 100644
--- a/dump_entry.hpp
+++ b/dump_entry.hpp
@@ -60,7 +60,7 @@
Entry(sdbusplus::bus::bus& bus, const std::string& objPath, uint32_t dumpId,
uint64_t timeStamp, uint64_t dumpSize, OperationStatus dumpStatus,
Manager& parent) :
- EntryIfaces(bus, objPath.c_str(), true),
+ EntryIfaces(bus, objPath.c_str(), EntryIfaces::action::emit_no_signals),
parent(parent), id(dumpId)
{
size(dumpSize);