Make dbus-sensors compile with clang-13
clang-13 finds a lot more warnings for unused variables than it used to,
and also picks up some warnings about errant std::moves that are in
place.
This commit fixes them.
Tested:
code compiles against clang-13
Signed-off-by: Ed Tanous <edtanous@google.com>
Change-Id: I80864287b7131acfe936c4b28afaf34ababb3029
diff --git a/src/Thresholds.cpp b/src/Thresholds.cpp
index bbe8e20..c52eee6 100644
--- a/src/Thresholds.cpp
+++ b/src/Thresholds.cpp
@@ -344,13 +344,13 @@
}
}
- if constexpr (debug)
+ // Throttle debug output, so that it does not continuously spam
+ ++cDebugThrottle;
+ if (cDebugThrottle >= 1000)
{
- // Throttle debug output, so that it does not continuously spam
- ++cDebugThrottle;
- if (cDebugThrottle >= 1000)
+ cDebugThrottle = 0;
+ if constexpr (debug)
{
- cDebugThrottle = 0;
std::cerr << "checkThresholds: High T=" << cHiTrue
<< " F=" << cHiFalse << " M=" << cHiMidstate
<< ", Low T=" << cLoTrue << " F=" << cLoFalse