commit a4d2768c83c04b8f9a640858b7343c44e7deea6d
author Zev Weiss <zev@bewilderbeest.net> Tue Jul 19 15:30:36 2022 -0700
committer Ed Tanous <ed@tanous.net> Wed Jul 20 14:37:33 2022 +0000
tree b2e3310b0199571efb37170a91acba5ad8d6a86d
parent afd150439db5c1fcbd437b25636124b56f8ab754

Factor out getPowerState() helper function

This same pattern had been open-coded in many of the sensor daemons;
let's reduce the duplication a bit.

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

src/HwmonTempMain.cpp

diff --git a/src/HwmonTempMain.cpp b/src/HwmonTempMain.cpp
index 988e8b9..d1d50d6 100644
--- a/src/HwmonTempMain.cpp
+++ b/src/HwmonTempMain.cpp
@@ -400,14 +400,7 @@
                 }
             }
 
-            auto findPowerOn = baseConfigMap.find("PowerState");
-            PowerState readState = PowerState::always;
-            if (findPowerOn != baseConfigMap.end())
-            {
-                std::string powerState =
-                    std::visit(VariantToStringVisitor(), findPowerOn->second);
-                setReadState(powerState, readState);
-            }
+            PowerState readState = getPowerState(baseConfigMap);
 
             auto permitSet = getPermitSet(baseConfigMap);
             auto& sensor = sensors[sensorName];