From 1fcce70caa2d7702c2b677a7720439b4c1270ab3 Mon Sep 17 00:00:00 2001 From: David Gibson Date: Mon, 26 Sep 2022 20:43:42 +1000 Subject: [PATCH] clang-tidy: Disable 'readability-identifier-length' This check complains about any identifier of less than 3 characters. For locals and parameters this is often pointlessly verbose. Disable it. Signed-off-by: David Gibson Signed-off-by: Stefano Brivio --- Makefile | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/Makefile b/Makefile index 42bc0ac..a39de07 100644 --- a/Makefile +++ b/Makefile @@ -237,6 +237,10 @@ docs: README.md # # - concurrency-mt-unsafe # TODO: check again if multithreading is implemented +# +# - readability-identifier-length +# Complains about any identifier <3 characters, reasonable for +# globals, pointlessly verbose for locals and parameters. clang-tidy: $(SRCS) $(HEADERS) clang-tidy -checks=*,-modernize-*,\ @@ -260,7 +264,8 @@ clang-tidy: $(SRCS) $(HEADERS) -bugprone-easily-swappable-parameters,\ -readability-function-cognitive-complexity,\ -altera-struct-pack-align,\ - -concurrency-mt-unsafe \ + -concurrency-mt-unsafe,\ + -readability-identifier-length \ -config='{CheckOptions: [{key: bugprone-suspicious-string-compare.WarnOnImplicitComparison, value: "false"}]}' \ --warnings-as-errors=* $(SRCS) -- $(filter-out -pie,$(FLAGS) $(CFLAGS))