Improve Macro Safety and Casting Practices: remove trailing semicolon

facebookincubator · Dec 17, 2024 · 186f320 · 186f320
1 parent 4cffa4b
commit 186f320
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/velox/common/base/SpillStats.cpp b/velox/common/base/SpillStats.cpp
@@ -134,7 +134,7 @@ bool SpillStats::operator<(const SpillStats& other) const {
     } else if (counter > other.counter) { \
       ++gtCount;                          \
     }                                     \
-  } while (0);
+  } while (0)
 
   UPDATE_COUNTER(spillRuns);
   UPDATE_COUNTER(spilledInputBytes);

diff --git a/velox/core/Metaprogramming.h b/velox/core/Metaprogramming.h
@@ -151,7 +151,7 @@ struct has_method {
         -> decltype(std::declval<__T>().MethodName(args...)) { \
       return {};                                               \
     }                                                          \
-  };
+  }
 
 // Calling Name::resolve<T>::type will return T::TypeName if T::TypeName
 // exists, and otherwise will return T::OtherTypeName (it's existence is not
@@ -169,5 +169,5 @@ struct has_method {
         std::void_t<decltype(sizeof(typename __T::TypeName))>> {     \
       using type = typename __T::TypeName;                           \
     };                                                               \
-  };
+  }
 } // namespace facebook::velox::util