commit 62bafc01bf4f4e2017d4a9e8d6d053f24ca30563
author Patrick Williams <patrick@stwcx.xyz> Thu Sep 08 17:35:35 2022 -0500
committer Ed Tanous <ed@tanous.net> Fri Sep 09 14:33:09 2022 +0000
tree 4f9760fba8d6978f35fd844fb4654f8d906733b0
parent 2e30bc2dfc54ea81f4b5e537b233f8ab50967109

clang-tidy: fix misc warnings

The following error reports have started to be reported by clang-tidy:

  * readability-qualified-auto - add 'const' to `auto&` iterators
  * bugprone-use-after-move - add break in loop after element is found

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I5314559f62f58aa032d4c74946b8e3e4ce6be808

include/openbmc_dbus_rest.hpp

diff --git a/include/openbmc_dbus_rest.hpp b/include/openbmc_dbus_rest.hpp
index d8a4e66..95503d7 100644
--- a/include/openbmc_dbus_rest.hpp
+++ b/include/openbmc_dbus_rest.hpp
@@ -1768,7 +1768,7 @@
                             }
                             else
                             {
-                                for (auto& prop : properties.items())
+                                for (const auto& prop : properties.items())
                                 {
                                     // if property name is empty, or
                                     // matches our search query, add it