From 80f7ff29964f16a9eef107c6c1d0b2c0554deab0 Mon Sep 17 00:00:00 2001 From: David Gibson Date: Tue, 14 May 2024 00:57:58 +1000 Subject: [PATCH] clang-tidy: Suppress macro to enum conversion warnings clang-tidy 18.1.1 in Fedora 40 complains about every #define of an integral value, suggesting it be converted to an enum. Although that's certainly possible, it's of dubious value and results in some awkward arrangements on out codebase. Suppress it globally. Signed-off-by: David Gibson Signed-off-by: Stefano Brivio --- Makefile | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/Makefile b/Makefile index c1e1f06..8ea1757 100644 --- a/Makefile +++ b/Makefile @@ -257,6 +257,12 @@ docs: README.md # we use. That sounds nice, but means it will often want a OS # specific header instead of a mostly standard one, such as # instead of . +# +# - cppcoreguidelines-macro-to-enum +# Want to replace all #defines of integers with enums. Kind of +# makes sense when those defines form an enum-like set, but +# weird for cases like standalone constants, and causes other +# awkwardness for a bunch of cases we use clang-tidy: $(SRCS) $(HEADERS) clang-tidy -checks=*,-modernize-*,\ @@ -283,7 +289,8 @@ clang-tidy: $(SRCS) $(HEADERS) -altera-struct-pack-align,\ -concurrency-mt-unsafe,\ -readability-identifier-length,\ - -misc-include-cleaner \ + -misc-include-cleaner,\ + -cppcoreguidelines-macro-to-enum \ -config='{CheckOptions: [{key: bugprone-suspicious-string-compare.WarnOnImplicitComparison, value: "false"}]}' \ --warnings-as-errors=* $(SRCS) -- $(filter-out -pie,$(FLAGS) $(CFLAGS) $(CPPFLAGS)) -DCLANG_TIDY_58992