AK: Suppress false-positive clang-tidy warning in Assertions.h

The definition of VERIFY_NOT_REACHED() as `assert(false)` causes the tool to suggest converting it to a static_assert. Which doesn't make any sense in context for what the macro is trying to do: crash the program at runtime.
Author: https://github.com/ADKaster Commit: https://github.com/SerenityOS/serenity/commit/7e2ee2e7252 Pull-request: https://github.com/SerenityOS/serenity/pull/10737 Reviewed-by: https://github.com/BenWiederhake Reviewed-by: https://github.com/PeterBindels-TomTom Reviewed-by: https://github.com/awesomekling Reviewed-by: https://github.com/dascandy Reviewed-by: https://github.com/trflynn89
2021-10-30 15:55:49 -06:00 · 2021-10-30 15:55:49 -06:00 · 7e2ee2e725 · 2024-07-18 01:08:28 +09:00
commit 7e2ee2e725
parent 6f5b9cc031
1 changed files with 1 additions and 1 deletions
--- a/AK/Assertions.h
+++ b/AK/Assertions.h
@ -11,7 +11,7 @@
 #else
 #    include <assert.h>
 #    define VERIFY assert
-#    define VERIFY_NOT_REACHED() assert(false)
+#    define VERIFY_NOT_REACHED() assert(false) /* NOLINT(cert-dcl03-c,misc-static-assert) No, this can't be static_assert, it's a runtime check */
 static constexpr bool TODO = false;
 #    define TODO() VERIFY(TODO)
 #endif