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/virtual_media.hpp

diff --git a/redfish-core/lib/virtual_media.hpp b/redfish-core/lib/virtual_media.hpp
index 844b60d..a682486 100644
--- a/redfish-core/lib/virtual_media.hpp
+++ b/redfish-core/lib/virtual_media.hpp
@@ -223,8 +223,9 @@
     BMCWEB_LOG_DEBUG << "Get Virtual Media resource data.";
 
     crow::connections::systemBus->async_method_call(
-        [resName, name, aResp](const boost::system::error_code ec,
-                               dbus::utility::ManagedObjectType& subtree) {
+        [resName, name,
+         aResp](const boost::system::error_code ec,
+                const dbus::utility::ManagedObjectType& subtree) {
             if (ec)
             {
                 BMCWEB_LOG_DEBUG << "DBUS response error";
@@ -232,7 +233,7 @@
                 return;
             }
 
-            for (auto& item : subtree)
+            for (const auto& item : subtree)
             {
                 std::string thispath = item.first.filename();
                 if (thispath.empty())