Improve clang-tidy

Author: IAmNotHanni

.clang-tidy

@@ -1,40 +1,27 @@
-Checks: >
-  clang-analyzer-*, 
-  bugprone-*, 
-  performance-*, 
-  readability-*, 
-  modernize-*, 
-  cppcoreguidelines-*, 
-  misc-*, 
-  -modernize-use-trailing-return-type, 
-  -modernize-use-auto, 
-  -modernize-use-using, 
-  -modernize-use-nodiscard, 
-  -cppcoreguidelines-macro-usage, 
-  -cppcoreguidelines-pro-bounds-pointer-arithmetic, 
-  -cppcoreguidelines-pro-type-union-access, 
-  -readability-function-cognitive-complexity, 
-  -llvmlibc-restrict-system-libc-headers,
-  -llvmlibc-implementation-in-namespace,
-  -altera-unroll-loops,
-  -misc-definitions-in-headers,
-  -readability-identifier-naming,
-  -readability-identifier-length
+Checks:
+  - '*'
+
+FormatStyle: 'file'
+HeaderFilterRegex: '^(?!.*vulkan-renderer/build).*'
 
-WarningsAsErrors: ''
-HeaderFilterRegex: '.*\.(cpp|h)$'
-FormatStyle: file
-AnalyzeTemporaryDtors: true
 CheckOptions:
-  - key:             modernize-loop-convert.MaxCopySize
-    value:           '16'
-  - key:             readability-identifier-naming.VariableCase
-    value:           camelBack
-  - key:             readability-identifier-naming.ClassCase
-    value:           CamelCase
-  - key:             readability-identifier-naming.FunctionCase
-    value:           camelBack
-  - key:             cppcoreguidelines-avoid-magic-numbers.IgnoreEnums
-    value:           '1'
-  - key:             cppcoreguidelines-avoid-magic-numbers.IgnoreOctalLiterals
-    value:           '1'
+  - key: 'bugprone-argument-comment.CommentBoolLiterals'
+    value: '0'
+  - key: 'bugprone-assert-side-effect.AssertMacros'
+    value: 'assert'
+  - key: 'bugprone-dangling-handle.HandleClasses'
+    value: 'std::basic_string_view;std::experimental::basic_string_view'
+  - key: 'bugprone-dynamic-static-initializers.HeaderFileExtensions'
+    value: ',h,hh,hpp,hxx'
+  - key: 'bugprone-suspicious-string-compare.WarnOnImplicitComparison'
+    value: '1'
+  - key: 'cppcoreguidelines-explicit-virtual-functions.IgnoreDestructors'
+    value: '1'
+  - key: 'cppcoreguidelines-no-malloc.Allocations'
+    value: '::malloc;::calloc'
+  - key: 'google-readability-function-size.StatementThreshold'
+    value: '800'
+  - key: 'readability-identifier-naming.FunctionCase'
+    value: 'lower_case'
+  - key: 'readability-static-accessed-through-instance.NameSpecifierNestingThreshold'
+    value: '3'

.github/workflows/static_code_analysis.yaml

@@ -15,7 +15,7 @@ jobs:
     - name: Install dependencies
       run: |
         sudo apt update
-        sudo apt install -y clang-15 clang-tidy-15 cmake ninja-build libc++-15-dev libc++abi-15-dev
+        sudo apt install -y clang-15 clang-tidy-15 cmake parallel libc++-15-dev libc++abi-15-dev
 
     - name: Set clang-tidy version
       run: echo "CLANG_TIDY=clang-tidy-15" >> $GITHUB_ENV
@@ -40,6 +40,24 @@ jobs:
           -DCMAKE_C_COMPILER=clang-15 \
           -DVMA_BUILD_SAMPLES=YES
 
-    - name: Run Clang-Tidy
+    - name: Check the files found for clang-tidy
       run: |
-        find . -path ./build -prune -o \( -name '*.cpp' -o -name '*.h' \) -print | xargs clang-tidy-15 -p build || true
+        find src include \
+          -path '*/_deps/*' -prune -o \
+          -path '*/build/*' -prune -o \
+          \( -name '*.cpp' -o -name '*.hpp' \) -print
+
+    - name: Run clang-tidy
+      run: |
+        find src include \
+          -path '*/_deps/*' -prune -o \
+          -path '*/build/*' -prune -o \
+          \( -name '*.cpp' -o -name '*.hpp' \) -print0 |
+          parallel -0 clang-tidy -p build {} |
+          tee output || true
+
+    - name: Summarize clang-tidy warnings
+      run: |
+        grep -hEo '\[[a-z0-9]+-[a-z0-9-]+\]' output \
+          | sort | uniq -c | sort -nr \
+          | sed 's/[][]//g' || true