Merge "fix(cppcheck): cppcheck ignore external repos" into integration

TF-RMM · Dec 15, 2022 · e2eea7c · e2eea7c
2 parents 6ef4ab3 + 204e7d2
commit e2eea7c
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 6 deletions.
diff --git a/tools/cppcheck/CMakeLists.txt b/tools/cppcheck/CMakeLists.txt
@@ -83,8 +83,14 @@ if(RMM_STATIC_ANALYSIS_CPPCHECK)
     list(APPEND cppcheck-flags "-DCHAR_BIT=8")
 
     #
-    # Suppress files or directories we don't want to receive warnings about. If
-    # you want to suppress specific files without using an inline suppression,
+    # Exclude files or directories we don't want to receive warnings about.
+    #
+
+    list(APPEND cppcheck-flags "-i${CMAKE_SOURCE_DIR}/ext/")
+    list(APPEND cppcheck-flags "-i${CMAKE_SOURCE_DIR}/lib/libc")
+
+    #
+    # If you want to suppress specific files without using an inline suppression,
     # do it in `suppressions.txt`.
     #
 

diff --git a/tools/cppcheck/suppressions.txt b/tools/cppcheck/suppressions.txt
@@ -12,7 +12,3 @@ unusedFunction
 // the `missingIncludeSystem` suppression was useless. It's right, but it's
 // also not particularly helpful.
 unmatchedSuppression
-
-// We don't want to receive warnings about code we don't control, which includes
-// any external libraries we vendor.
-*:ext/*