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 <[email protected]>

Author: Pavithrab7

.clang-tidy

@@ -243,6 +243,7 @@ performance-type-promotion-in-math-fn,
 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,

bmc_state_manager.cpp

@@ -141,7 +141,9 @@ void BMC::executeTransition(const Transition tranReq)
         // 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;
 

scheduled_host_transition.cpp

@@ -248,7 +248,7 @@ int ScheduledHostTransition::onTimeChange(sd_event_source* /* es */, int fd,
     // 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();