commit 9eb808c1a84a76e92b0e01fa95d3f160f38d7c3f
author Ed Tanous <edtanous@google.com> Tue Jan 25 10:19:23 2022 -0800
committer Ed Tanous <ed@tanous.net> Wed Feb 09 20:45:00 2022 +0000
tree f3631a3f2a97fb0f048fb1dd5d295b867e8d37fc
parent b3db89663256a462e9c90bf260d5058014765137

Enable readability-avoid-const-params-in-decls

This check involves explicitly declaring variables const when they're
declared auto, which helps in readability, and makes it more clear that
the variables are const.

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

include/openbmc_dbus_rest.hpp

diff --git a/include/openbmc_dbus_rest.hpp b/include/openbmc_dbus_rest.hpp
index 9233a27..8aab757 100644
--- a/include/openbmc_dbus_rest.hpp
+++ b/include/openbmc_dbus_rest.hpp
@@ -2232,7 +2232,7 @@
                 }
                 std::filesystem::directory_iterator files(loc);
 
-                for (auto& file : files)
+                for (const auto& file : files)
                 {
                     std::ifstream readFile(file.path());
                     if (!readFile.good())