diff options
author | Ed Tanous <edtanous@google.com> | 2023-02-16 21:37:01 +0300 |
---|---|---|
committer | Ed Tanous <ed@tanous.net> | 2023-02-24 20:21:40 +0300 |
commit | 5e7e2dc585dc7a7e62d2b648b541e7f50c39ea5d (patch) | |
tree | ee3993e0fb2dc6e244a5fc31c7e83b26783161ee /include/dbus_utility.hpp | |
parent | 26ccae32112679c4653c1e3f8a1203c828bea05c (diff) | |
download | bmcweb-5e7e2dc585dc7a7e62d2b648b541e7f50c39ea5d.tar.xz |
Take boost error_code by reference
By convention, we should be following boost here, and passing error_code
by reference, not by value. This makes our code consistent, and removes
the need for a copy in some cases.
Signed-off-by: Ed Tanous <edtanous@google.com>
Change-Id: Id42ea4a90b6685a84818b87d1506c11256b3b9ae
Diffstat (limited to 'include/dbus_utility.hpp')
-rw-r--r-- | include/dbus_utility.hpp | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/include/dbus_utility.hpp b/include/dbus_utility.hpp index e953a9f0eb..f41c422d30 100644 --- a/include/dbus_utility.hpp +++ b/include/dbus_utility.hpp @@ -133,7 +133,7 @@ inline void checkDbusPathExists(const std::string& path, Callback&& callback) { crow::connections::systemBus->async_method_call( [callback{std::forward<Callback>(callback)}]( - const boost::system::error_code ec, + const boost::system::error_code& ec, const dbus::utility::MapperGetObject& objectNames) { callback(!ec && !objectNames.empty()); }, @@ -151,7 +151,7 @@ inline void { crow::connections::systemBus->async_method_call( [callback{std::move(callback)}]( - const boost::system::error_code ec, + const boost::system::error_code& ec, const MapperGetSubTreeResponse& subtree) { callback(ec, subtree); }, "xyz.openbmc_project.ObjectMapper", "/xyz/openbmc_project/object_mapper", @@ -167,7 +167,7 @@ inline void getSubTreePaths( { crow::connections::systemBus->async_method_call( [callback{std::move(callback)}]( - const boost::system::error_code ec, + const boost::system::error_code& ec, const MapperGetSubTreePathsResponse& subtreePaths) { callback(ec, subtreePaths); }, |