commit f254ba77a5abf5a806efb35055b5e0d4ced40251
author Ed Tanous <ed.tanous@intel.com> Fri Oct 12 13:40:35 2018 -0700
committer Ed Tanous <ed.tanous@intel.com> Fri Oct 12 20:41:16 2018 +0000
tree e7a2f6dd0f61eacba3c7c289c150e8e30e622dd6
parent ca537928fbd8c22736a7cd30c98d5f3640a3523a

Avoid some copies in some loops

Auto does not work the way you want it to in this case, and gives you
copies, when you really wanted const references.  This commit moves the
loops to const references.

Change-Id: I7aceedc03528b4d41c56b100e0c956a64b001ad9
Signed-off-by: Ed Tanous <ed.tanous@intel.com>

include/openbmc_dbus_rest.hpp

diff --git a/include/openbmc_dbus_rest.hpp b/include/openbmc_dbus_rest.hpp
index e0d9986..110e6ac 100644
--- a/include/openbmc_dbus_rest.hpp
+++ b/include/openbmc_dbus_rest.hpp
@@ -166,9 +166,9 @@
                 return;
             }
 
-            for (auto pathGroup : objects)
+            for (const auto &pathGroup : objects)
             {
-                for (auto connectionGroup : pathGroup.second)
+                for (const auto &connectionGroup : pathGroup.second)
                 {
                     if (connectionGroup.first == connection_name)
                     {