diff options
| author | Max Wipfli <mail@maxwipfli.ch> | 2022-02-21 17:34:19 +0100 |
|---|---|---|
| committer | Idan Horowitz <idan.horowitz@gmail.com> | 2022-02-21 19:01:16 +0200 |
| commit | d29d9462e9c34ff89990073deb7d7fadb66b1779 (patch) | |
| tree | 754f50a0b0f0a560b4fc74b1ccbc36a56824fb36 | |
| parent | 64489644852e6628d0be199ceb8f8b058f36f4d5 (diff) | |
| download | serenity-d29d9462e9c34ff89990073deb7d7fadb66b1779.zip | |
AK: Suppress clang-tidy warning on TODO()
This adds a NOLINT directive to the definition of the TODO() macro.
clang-tidy wants the assert replaced with a static_assert, since the
macro simply resolves to assert(false). This is obviously nonsensical,
since we want the code to still compile even with TODO().
The same fix has already been implemented for VERIFY_NOT_REACHED().
| -rw-r--r-- | AK/Assertions.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/AK/Assertions.h b/AK/Assertions.h index cb28d8f7f4..55e7c989da 100644 --- a/AK/Assertions.h +++ b/AK/Assertions.h @@ -13,5 +13,5 @@ # define VERIFY assert # 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) +# define TODO() VERIFY(TODO) /* NOLINT(cert-dcl03-c,misc-static-assert) No, this can't be static_assert, it's a runtime check */ #endif |