Refactor: Review and fix Wrap multistatement macros in a do-while loop

unix280 · Dec 12, 2024 · 243ee3d · 243ee3d
1 parent 41aba92
commit 243ee3d
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/presto-native-execution/presto_cpp/main/PeriodicTaskManager.cpp b/presto-native-execution/presto_cpp/main/PeriodicTaskManager.cpp
@@ -33,9 +33,11 @@
 
 namespace {
 #define REPORT_IF_NOT_ZERO(name, counter)   \
-  if ((counter) != 0) {                     \
+  do {                                      \
+    if ((counter) != 0) {                   \
     RECORD_METRIC_VALUE((name), (counter)); \
-  }
+    }                                       \
+  } while (0)
 } // namespace
 
 namespace facebook::presto {