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_interface_added`.
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: Id3689b24f8719bafa9e5a67960848edff4ddf8c5
diff --git a/version.hpp b/version.hpp
index 51a7024..7cd2be7 100644
--- a/version.hpp
+++ b/version.hpp
@@ -86,7 +86,8 @@
ItemUpdater& parent, const std::string& versionId,
const std::string& versionString, VersionPurpose versionPurpose,
const std::string& filePath, eraseFunc callback) :
- VersionInherit(bus, (objPath).c_str(), true),
+ VersionInherit(bus, (objPath).c_str(),
+ VersionInherit::action::defer_emit),
eraseCallback(callback), bus(bus), objPath(objPath), parent(parent),
versionId(versionId), versionStr(versionString),
chassisStateSignals(