commit 914e2d5d5dbeddca9f48504256f7089efe86fc65
author Ed Tanous <edtanous@google.com> Fri Jan 07 11:38:34 2022 -0800
committer Ed Tanous <ed@tanous.net> Mon Jan 10 17:36:54 2022 +0000
tree b9e60be35e5a15989b5d2d70038e9085dbd6e2bb
parent 711ac7a931dd3f151fc4064063b5ea90404b9054

Enforce const correctness

For all async calls, we should be consistently capturing non trivial
objects by const reference.  This corrects bmcweb to be consistent and
capture errors by const value, and objects by const reference.

Tested: Code compiles.  Trivial changes.

This saves about 300 bytes on our compressed binary size.

Signed-off-by: Ed Tanous <edtanous@google.com>
Change-Id: Ib3e0b6edef9803a1c480701556949488406305d4

redfish-core/lib/processor.hpp

diff --git a/redfish-core/lib/processor.hpp b/redfish-core/lib/processor.hpp
index cdeb4a0..1155000 100644
--- a/redfish-core/lib/processor.hpp
+++ b/redfish-core/lib/processor.hpp
@@ -832,7 +832,7 @@
                            const std::string& objPath)
 {
     crow::connections::systemBus->async_method_call(
-        [aResp](boost::system::error_code ec,
+        [aResp](const boost::system::error_code ec,
                 const OperatingConfigProperties& properties) {
             if (ec)
             {
@@ -1053,8 +1053,8 @@
 
     // Set the property, with handler to check error responses
     crow::connections::systemBus->async_method_call(
-        [resp, appliedConfigUri](boost::system::error_code ec,
-                                 sdbusplus::message::message& msg) {
+        [resp, appliedConfigUri](const boost::system::error_code ec,
+                                 const sdbusplus::message::message& msg) {
             handleAppliedConfigResponse(resp, appliedConfigUri, ec, msg);
         },
         *controlService, cpuObjectPath, "org.freedesktop.DBus.Properties",