commit 44bbf11d343ea3427fd8da74e84a6fa6976c34b9
author Pavithra Barithaya <pavithrabarithaya07@gmail.com> Fri Jun 21 11:43:30 2024 -0500
committer Pavithra Barithaya <pavithrabarithaya07@gmail.com> Fri Jul 05 14:49:33 2024 +0530
tree d1bfaea5a1ad7ec85b09e8ed34ab0a7069444bb8
parent 480d430e5096a7db94de0dbdf9f65d6b7255eb49

clang-tidy: Enable readability-simplify-boolean-expr check

This checks for boolean expressions involving boolean constants
and simplifies them to use the appropriate boolean expression
directly.

Change-Id: I4ad7ec4ddfa4cfe9a0cf0d569d3d81c478c1776a
Signed-off-by: Pavithra Barithaya <pavithrabarithaya07@gmail.com>

chassis_state_manager.cpp

diff --git a/chassis_state_manager.cpp b/chassis_state_manager.cpp
index 189f2de..05cc784 100644
--- a/chassis_state_manager.cpp
+++ b/chassis_state_manager.cpp
@@ -280,7 +280,7 @@
                     continue;
                 }
 
-                if (std::get<bool>(properties["IsPresent"]) != true)
+                if (!std::get<bool>(properties["IsPresent"]))
                 {
                     // There is a UPS detected but it is not officially
                     // "present" yet. Monitor it for state change.