From 0559a1a89c8258b87162238a842cb11ce31e24e3 Mon Sep 17 00:00:00 2001 From: Otto Moerbeek Date: Mon, 21 Oct 2024 09:52:16 +0200 Subject: [PATCH] Finally fix the clang-tidy issues. Root cause was the slightly different parsing or the Checks: line, newer version handle the single quotes differently. Also remove two entries that are not present on older clang-tidy implementations. --- .clang-tidy.bugs | 19 ++++++++++--------- .clang-tidy.full | 21 +++++++++++---------- 2 files changed, 21 insertions(+), 19 deletions(-) diff --git a/.clang-tidy.bugs b/.clang-tidy.bugs index 612bcf370c..f6499ffc38 100644 --- a/.clang-tidy.bugs +++ b/.clang-tidy.bugs @@ -1,17 +1,18 @@ --- FormatStyle: none -HeaderFileExtensions: - - h - - hh - - hpp -ImplementationFileExtensions: - - c - - cc - - cpp +# Older (pre 19) versions of clang-tidy do not grok these +# HeaderFileExtensions: +# - h +# - hh +# - hpp +#ImplementationFileExtensions: +# - c +# - cc +# - cpp WarningsAsErrors: '' HeaderFilterRegex: '' Checks: | - 'clang-diagnostic-*,clang-analyzer-*,bugprone-*,concurrency-*,-modernize-use-trailing-return-type,-readability-magic-numbers,-cppcoreguidelines-avoid-magic-numbers,-performance-avoid-endl' + clang-diagnostic-*,clang-analyzer-*,bugprone-*,concurrency-*,-modernize-use-trailing-return-type,-readability-magic-numbers,-cppcoreguidelines-avoid-magic-numbers,-performance-avoid-endl # yamllint disable rule:line-length CheckOptions: - key: bugprone-string-constructor.LargeLengthThreshold diff --git a/.clang-tidy.full b/.clang-tidy.full index 3c73e19b47..ff0eb73550 100644 --- a/.clang-tidy.full +++ b/.clang-tidy.full @@ -1,17 +1,18 @@ --- -FormatStyle: none -HeaderFileExtensions: - - h - - hh - - hpp -ImplementationFileExtensions: - - c - - cc - - cpp +FormatStyle: non +# Older (pre 19) versions of clang-tidy do not grok these +#HeaderFileExtensions: +# - h +# - hh +# - hpp +#ImplementationFileExtensions: +# - c +# - cc +# - cpp WarningsAsErrors: '' HeaderFilterRegex: '' Checks: | - 'clang-diagnostic-*,clang-analyzer-*,cppcoreguidelines-*,bugprone-*,concurrency-*,modernize-*,performance-*,portability-*,readability-*,-modernize-use-trailing-return-type,-readability-magic-numbers,-cppcoreguidelines-avoid-magic-numbers,-cppcoreguidelines-pro-type-union-access,-cppcoreguidelines-avoid-non-const-global-variables,-cppcoreguidelines-pro-type-vararg,-cppcoreguidelines-avoid-do-while,-cppcoreguidelines-avoid-const-or-ref-data-members,-performance-avoid-endl' + clang-diagnostic-*,clang-analyzer-*,cppcoreguidelines-*,bugprone-*,concurrency-*,modernize-*,performance-*,portability-*,readability-*,-modernize-use-trailing-return-type,-readability-magic-numbers,-cppcoreguidelines-avoid-magic-numbers,-cppcoreguidelines-pro-type-union-access,-cppcoreguidelines-avoid-non-const-global-variables,-cppcoreguidelines-pro-type-vararg,-cppcoreguidelines-avoid-do-while,-cppcoreguidelines-avoid-const-or-ref-data-members,-performance-avoid-endl # yamllint disable rule:line-length CheckOptions: - key: readability-suspicious-call-argument.PrefixSimilarAbove -- 2.47.2