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/ADCSensorMain.cpp

diff --git a/src/ADCSensorMain.cpp b/src/ADCSensorMain.cpp
index cec0984..fe8334d 100644
--- a/src/ADCSensorMain.cpp
+++ b/src/ADCSensorMain.cpp
@@ -253,14 +253,7 @@
                 }
             }
 
-            auto findPowerOn = baseConfiguration->second.find("PowerState");
-            PowerState readState = PowerState::always;
-            if (findPowerOn != baseConfiguration->second.end())
-            {
-                std::string powerState =
-                    std::visit(VariantToStringVisitor(), findPowerOn->second);
-                setReadState(powerState, readState);
-            }
+            PowerState readState = getPowerState(baseConfiguration->second);
 
             auto& sensor = sensors[sensorName];
             sensor = nullptr;