clang-tidy: Enable readability-braces-around-statements check
This checks that bodies of if statements and loops
(for, do while, and while) are inside braces.
Change-Id: Ia5924cc4e9dca98cc29bbaa50923e3269db22089
Signed-off-by: Pavithra Barithaya <pavithrabarithaya07@gmail.com>
diff --git a/.clang-tidy b/.clang-tidy
index 462941c..f063ed1 100644
--- a/.clang-tidy
+++ b/.clang-tidy
@@ -243,6 +243,7 @@
performance-unnecessary-copy-initialization,
performance-unnecessary-value-param,
readability-avoid-const-params-in-decls,
+readability-braces-around-statements,
readability-const-return-type,
readability-delete-null-pointer,
readability-deleted-default,
diff --git a/bmc_state_manager.cpp b/bmc_state_manager.cpp
index bbea272..c891376 100644
--- a/bmc_state_manager.cpp
+++ b/bmc_state_manager.cpp
@@ -141,7 +141,9 @@
// Check to make sure it can be found
auto iter = SYSTEMD_TABLE.find(tranReq);
if (iter == SYSTEMD_TABLE.end())
+ {
return;
+ }
const auto& sysdUnit = iter->second;
diff --git a/scheduled_host_transition.cpp b/scheduled_host_transition.cpp
index 077d794..01e698e 100644
--- a/scheduled_host_transition.cpp
+++ b/scheduled_host_transition.cpp
@@ -248,7 +248,7 @@
// We are not interested in the data here.
// So read until there is no new data here in the FD
while (read(fd, time.data(), time.max_size()) > 0)
- ;
+ {}
debug("BMC system time is changed");
schedHostTran->handleTimeUpdates();