enabled some clang-tidy warnings for non-GUI code (#5679)

There are some clang-tidy warnings which cannot properly handle the
extensions used in the Qt code so the produce lots of false positives
and were previously enabled. With clang-tidy now offering the
possibility to inherit the parent configuration we can enable these
across the project and simply disable them for the GUI code.

No additional warnings were found as they were already fixed in earlier
commits without enabling the checks in question.
This commit is contained in:
Oliver Stöneberg 2023-11-20 10:06:46 +01:00 committed by GitHub
parent 92c089d4ad
commit 3bafe164a3
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
4 changed files with 21 additions and 11 deletions

View File

@ -61,7 +61,6 @@ Checks: >
-readability-braces-around-statements,
-readability-const-return-type,
-readability-container-data-pointer,
-readability-convert-member-functions-to-static,
-readability-function-cognitive-complexity,
-readability-function-size,
-readability-identifier-length,
@ -69,7 +68,6 @@ Checks: >
-readability-implicit-bool-conversion,
-readability-isolate-declaration,
-readability-magic-numbers,
-readability-redundant-access-specifiers,
-readability-suspicious-call-argument,
-readability-uppercase-literal-suffix
WarningsAsErrors: '*'

View File

@ -71,10 +71,6 @@ These might change the behavior of code which might not be intended (need to fil
This leads to a mismatch of raw string literals and regular ones and does reduce the readability.
`readability-convert-member-functions-to-static`<br/>
Disabled because of false positives with Qt `slot` methods (see https://github.com/llvm/llvm-project/issues/57520).
`-clang-analyzer-*`<br/>
Disabled because of false positives (needs to file an upstream bug report).
@ -112,10 +108,6 @@ Produces a lot of false positives since it is too vague in its analysis.
Produces warnings which might be considered false positives starting with C++11 - see https://github.com/llvm/llvm-project/issues/54526.
`readability-redundant-access-specifiers`<br/>
Reports warning with the Qt `<access-specifier> slots:` syntax in class declarations - see https://github.com/llvm/llvm-project/issues/60055.
`modernize-avoid-c-arrays`<br/>
Produces warnings when `const char[]` is being used which is quite common in our code. Does not make sense to enable before C++17 when `std::string_view` becomes available.
@ -197,3 +189,13 @@ This is the most expensive check for several files and it is providing much in t
`modernize-use-nullptr`
This is already covered by the `-Wzero-as-null-pointer-constant` compiler warning so there is no need for an additional check.
### Disabled for GUI only
`readability-convert-member-functions-to-static`<br/>
Disabled because of false positives with Qt `slot` methods (see https://github.com/llvm/llvm-project/issues/57520).
`readability-redundant-access-specifiers`<br/>
Reports warning with the Qt `<access-specifier> slots:` syntax in class declarations - see https://github.com/llvm/llvm-project/issues/60055.

5
gui/.clang-tidy Normal file
View File

@ -0,0 +1,5 @@
---
Checks: >
-readability-convert-member-functions-to-static,
-readability-redundant-access-specifiers
InheritParentConfig: true

5
tools/triage/.clang-tidy Normal file
View File

@ -0,0 +1,5 @@
---
Checks: >
-readability-convert-member-functions-to-static,
-readability-redundant-access-specifiers
InheritParentConfig: true