commit 30eb03f3d5054088727f3065e367266b2a82b31d
author Ed Tanous <edtanous@google.com> Wed Jan 26 16:51:00 2022 -0800
committer Ed Tanous <ed@tanous.net> Mon Jan 31 20:55:07 2022 +0000
tree da98609ced2b711a3cbb8f420336f4b96808c8e6
parent f69fbf998745bf006748c3b6ba6558443b560d21

Remove redundant empty check

Checking a vector for empty, then immediately iterating it doesn't
change anything.  An empty vector will just return immediately.

Tested: No-OP

Signed-off-by: Ed Tanous <edtanous@google.com>
Change-Id: Ibf8b3e5c6481bbd17d24b9b5a43320ebac4b79f4

src/Thresholds.cpp

diff --git a/src/Thresholds.cpp b/src/Thresholds.cpp
index b82a511..f50905a 100644
--- a/src/Thresholds.cpp
+++ b/src/Thresholds.cpp
@@ -212,11 +212,6 @@
 
 void updateThresholds(Sensor* sensor)
 {
-    if (sensor->thresholds.empty())
-    {
-        return;
-    }
-
     for (const auto& threshold : sensor->thresholds)
     {
         if (!findOrder(threshold.level, threshold.direction))