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/dbus_monitor.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/dbus_monitor.hpp')
| -rw-r--r-- | include/dbus_monitor.hpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/include/dbus_monitor.hpp b/include/dbus_monitor.hpp index 129422e4ae..5f409d7d70 100644 --- a/include/dbus_monitor.hpp +++ b/include/dbus_monitor.hpp @@ -84,7 +84,7 @@ inline int onPropertyUpdate(sd_bus_message* m, void* userdata, } // data is type oa{sa{sv}} which is an array[2] of string, object - for (auto& entry : data[1].items()) + for (const auto& entry : data[1].items()) { auto it = thisSession->second.interfaces.find(entry.key()); if (it != thisSession->second.interfaces.end()) |