Remove isValidLevel()
Every usage of isValidLevel is done after the levels have been parsed
and are already known to be correct. Checking that Level is valid is
wasteful, and in some cases, is already checked by
getThresholdInterface, which can already bounds check this.
This check is not needed.
Signed-off-by: Ed Tanous <edtanous@google.com>
Change-Id: I1d03c1c66ae37ca86a48175ec9c739c2d3966662
diff --git a/include/sensor.hpp b/include/sensor.hpp
index e966c20..12de959 100644
--- a/include/sensor.hpp
+++ b/include/sensor.hpp
@@ -261,10 +261,7 @@
{
threshold.hysteresis = hysteresisTrigger;
}
- if (!thresholds::isValidLevel(threshold.level))
- {
- continue;
- }
+
std::shared_ptr<sdbusplus::asio::dbus_interface> iface =
getThresholdInterface(threshold.level);