clang-tidy: add NOLInt to macro usage and the errors in there

OpenBrickProtocolFoundation · May 24, 2024 · f871505 · f871505
1 parent 92095fc
commit f871505
Showing 1 changed file with 7 additions and 4 deletions.
diff --git a/src/helper/utils.hpp b/src/helper/utils.hpp
@@ -120,10 +120,13 @@ namespace utils {
 #define UNUSED(x) (void(x)) // NOLINT(cppcoreguidelines-macro-usage)
 
 #if !defined(NDEBUG)
-#define UNREACHABLE() /* NOLINT(cppcoreguidelines-macro-usage)*/                                           \
-    do {                                                                                                   \
-        std::cerr << "UNREACHABLE " << (__FILE__) << ":" << (__LINE__) << " - " << (__FUNCTION__) << "\n"; \
-        utils::unreachable();                                                                              \
+#define UNREACHABLE()                             /* NOLINT(cppcoreguidelines-macro-usage)*/                       \
+    do {                                          /* NOLINT(cppcoreguidelines-avoid-do-while)*/                    \
+        std::cerr << "UNREACHABLE " << (__FILE__) /* NOLINT(cppcoreguidelines-pro-bounds-array-to-pointer-decay)*/ \
+                  << ":" << (__LINE__) << " - "                                                                    \
+                  << (__FUNCTION__) /* NOLINT(cppcoreguidelines-pro-bounds-array-to-pointer-decay)*/               \
+                  << "\n";                                                                                         \
+        utils::unreachable();                                                                                      \
     } while (false)
 #else
 #define UNREACHABLE() utils::unreachable // NOLINT(cppcoreguidelines-macro-usage)