Skip to content

Commit 511ed95

Browse files
committed
run clang-tidy with C++20
1 parent cc21af7 commit 511ed95

3 files changed

Lines changed: 16 additions & 4 deletions

File tree

.clang-tidy

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -43,15 +43,20 @@ Checks: >
4343
-modernize-deprecated-ios-base-aliases,
4444
-misc-include-cleaner,
4545
-misc-unused-using-decls,
46+
-modernize-avoid-bind,
4647
-modernize-loop-convert,
4748
-modernize-macro-to-enum,
49+
-modernize-make-unique,
4850
-modernize-raw-string-literal,
4951
-modernize-replace-auto-ptr,
5052
-modernize-return-braced-init-list,
5153
-modernize-type-traits,
54+
-modernize-use-constraints,
5255
-modernize-use-designated-initializers,
5356
-modernize-use-nodiscard,
57+
-modernize-use-ranges,
5458
-modernize-use-scoped-lock,
59+
-modernize-use-starts-ends-with,
5560
-modernize-use-trailing-return-type,
5661
-performance-avoid-endl,
5762
-performance-inefficient-string-concatenation,
@@ -61,6 +66,7 @@ Checks: >
6166
-portability-std-allocator-const,
6267
-readability-avoid-nested-conditional-operator,
6368
-readability-braces-around-statements,
69+
-readability-container-contains,
6470
-readability-container-data-pointer,
6571
-readability-enum-initial-value,
6672
-readability-function-cognitive-complexity,

.github/workflows/clang-tidy.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -61,7 +61,7 @@ jobs:
6161
6262
- name: Prepare CMake
6363
run: |
64-
cmake -S . -B cmake.output -G "Unix Makefiles" -DHAVE_RULES=On -DBUILD_TESTS=On -DBUILD_GUI=On -DWITH_QCHART=On -DBUILD_TRIAGE=On -DENABLE_CHECK_INTERNAL=On -DCMAKE_GLOBAL_AUTOGEN_TARGET=On -DDISABLE_DMAKE=On -DCPPCHK_GLIBCXX_DEBUG=Off -DWARNINGS_ARE_ERRORS=On
64+
cmake -S . -B cmake.output -G "Unix Makefiles" -DCMAKE_CXX_STANDARD=20 -DHAVE_RULES=On -DBUILD_TESTS=On -DBUILD_GUI=On -DWITH_QCHART=On -DBUILD_TRIAGE=On -DENABLE_CHECK_INTERNAL=On -DCMAKE_GLOBAL_AUTOGEN_TARGET=On -DDISABLE_DMAKE=On -DCPPCHK_GLIBCXX_DEBUG=Off -DWARNINGS_ARE_ERRORS=On
6565
env:
6666
CC: clang-22
6767
CXX: clang++-22

clang-tidy.md

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -133,6 +133,7 @@ Does not improve the readability.
133133
`misc-unconventional-assign-operator`<br/>
134134
`bugprone-throwing-static-initialization`<br/>
135135
`bugprone-command-processor`<br/>
136+
`modernize-use-nodiscard`<br/>
136137

137138
To be evaluated (need to remove exclusion).
138139

@@ -150,10 +151,15 @@ To be evaluated (need to remove exclusion).
150151
To be evaluated (need to enable explicitly).
151152

152153
`modernize-type-traits`<br/>
153-
`modernize-use-nodiscard`<br/>
154154
`modernize-use-scoped-lock`<br/>
155-
156-
These apply to codebases which use later standards then C++11 (C++17 is used when building with Qt6) so we cannot simply apply them.
155+
`modernize-use-constraints`<br/>
156+
`modernize-use-starts-ends-with`<br/>
157+
`modernize-avoid-bind`<br/>
158+
`modernize-make-unique`<br/>
159+
`modernize-use-ranges`<br/>
160+
`readability-container-contains`<br/>
161+
162+
These require us to default to a later standard than C++11 so we cannot apply them.
157163

158164
`portability-avoid-pragma-once`<br/>
159165

0 commit comments

Comments
 (0)