diff options
author | Patrick Williams <patrick@stwcx.xyz> | 2022-09-09 01:35:35 +0300 |
---|---|---|
committer | Ed Tanous <ed@tanous.net> | 2022-09-09 17:33:09 +0300 |
commit | 62bafc01bf4f4e2017d4a9e8d6d053f24ca30563 (patch) | |
tree | 4f9760fba8d6978f35fd844fb4654f8d906733b0 /include/openbmc_dbus_rest.hpp | |
parent | 2e30bc2dfc54ea81f4b5e537b233f8ab50967109 (diff) | |
download | bmcweb-62bafc01bf4f4e2017d4a9e8d6d053f24ca30563.tar.xz |
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
Diffstat (limited to 'include/openbmc_dbus_rest.hpp')
-rw-r--r-- | include/openbmc_dbus_rest.hpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/include/openbmc_dbus_rest.hpp b/include/openbmc_dbus_rest.hpp index d8a4e66d6f..95503d706e 100644 --- a/include/openbmc_dbus_rest.hpp +++ b/include/openbmc_dbus_rest.hpp @@ -1768,7 +1768,7 @@ inline void handleGet(const std::shared_ptr<bmcweb::AsyncResp>& asyncResp, } else { - for (auto& prop : properties.items()) + for (const auto& prop : properties.items()) { // if property name is empty, or // matches our search query, add it |