On Sat, Sep 24, 2022 at 01:08:55AM +0200, Stefano Brivio wrote:On Fri, 23 Sep 2022 17:20:34 +1000 David Gibson <david(a)gibson.dropbear.id.au> wrote:I am respinning, so I've made this change.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 <david(a)gibson.dropbear.id.au> --- Makefile | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/Makefile b/Makefile index 432ee7a..0598865 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,7 @@ 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 \I would rather keep those (unrelated) switches on different lines, I find it's a bit easier to follow through the comments like that. I can change this on merge if you agree, unless you respin.I reviewed the rest of the series, it looks great to me, I have no further observations -- but I haven't tested it yet.-- David Gibson | I'll have my music baroque, and my code david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_ | _way_ _around_! http://www.ozlabs.org/~dgibson