commit a18083303cefaca1bb30e4f80e3414bb4cdb0e39
author Zev Weiss <zev@bewilderbeest.net> Fri Aug 12 18:21:01 2022 -0700
committer Zev Weiss <zev@bewilderbeest.net> Mon Aug 15 19:24:57 2022 -0700
tree 417b81e0ef740b4fef3fc61768ac1925e0a20896
parent 77636ecb4a471bcf3a379fd64c85101c5b93bb65

HwmonTempMain: Replace iterator pairs with structured bindings

Signed-off-by: Zev Weiss <zev@bewilderbeest.net>
Change-Id: If1e75f992fbb1f1729102003a06859db2bca94c9

src/HwmonTempMain.cpp

diff --git a/src/HwmonTempMain.cpp b/src/HwmonTempMain.cpp
index 5c9ea10..47f2f07 100644
--- a/src/HwmonTempMain.cpp
+++ b/src/HwmonTempMain.cpp
@@ -185,13 +185,10 @@
     buildSensorConfigMap(const ManagedObjectType& sensorConfigs)
 {
     SensorConfigMap configMap;
-    for (const std::pair<sdbusplus::message::object_path, SensorData>& sensor :
-         sensorConfigs)
+    for (const auto& [path, cfgData] : sensorConfigs)
     {
-        for (const std::pair<std::string, SensorBaseConfigMap>& cfgmap :
-             sensor.second)
+        for (const auto& [intf, cfg] : cfgData)
         {
-            const SensorBaseConfigMap& cfg = cfgmap.second;
             auto busCfg = cfg.find("Bus");
             auto addrCfg = cfg.find("Address");
             if ((busCfg == cfg.end()) || (addrCfg == cfg.end()))
@@ -202,7 +199,7 @@
             if ((std::get_if<uint64_t>(&busCfg->second) == nullptr) ||
                 (std::get_if<uint64_t>(&addrCfg->second) == nullptr))
             {
-                std::cerr << sensor.first.str << " Bus or Address invalid\n";
+                std::cerr << path.str << " Bus or Address invalid\n";
                 continue;
             }
 
@@ -227,16 +224,14 @@
 
             SensorConfigKey key = {std::get<uint64_t>(busCfg->second),
                                    std::get<uint64_t>(addrCfg->second)};
-            SensorConfig val = {sensor.first.str, sensor.second, cfgmap.first,
-                                cfg, hwmonNames};
+            SensorConfig val = {path.str, cfgData, intf, cfg, hwmonNames};
 
             auto [it, inserted] = configMap.emplace(key, std::move(val));
             if (!inserted)
             {
-                std::cerr << sensor.first.str
-                          << ": ignoring duplicate entry for {" << key.bus
-                          << ", 0x" << std::hex << key.addr << std::dec
-                          << "}\n";
+                std::cerr << path.str << ": ignoring duplicate entry for {"
+                          << key.bus << ", 0x" << std::hex << key.addr
+                          << std::dec << "}\n";
             }
         }
     }