StrixOS/build
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "Add new clang and clang-tidy warning patterns"

Browse Source
This commit is contained in:
Treehugger Robot
2021-09-23 23:31:01 +00:00
committed by Gerrit Code Review
parent 756e53089a 3bb6c9448b
commit 84a87eb9ed
2 changed files with 10 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
tools/warn/cpp_warn_patterns.py
View File

@@ -93,6 +93,8 @@ warn_patterns = [
[r".*: warning: Null passed to a callee that requires a non-null"]), [r".*: warning: Null passed to a callee that requires a non-null"]),
medium('Unused command line argument', medium('Unused command line argument',
[r".*: warning: argument unused during compilation: .+"]), [r".*: warning: argument unused during compilation: .+"]),
medium('Set but not used',
[r".*: warning: .* set but not used.*Wunused-but-set"]),
medium('Unused parameter', medium('Unused parameter',
[r".*: warning: unused parameter '.*'"]), [r".*: warning: unused parameter '.*'"]),
medium('Unused function, variable, label, comparison, etc.', medium('Unused function, variable, label, comparison, etc.',

10
tools/warn/tidy_warn_patterns.py
View File

@@ -23,15 +23,19 @@ from .cpp_warn_patterns import compile_patterns
from .severity import Severity from .severity import Severity
def tidy_warn_pattern(description, pattern): def tidy_warn(description, patterns):
return { return {
'category': 'C/C++', 'category': 'C/C++',
'severity': Severity.TIDY, 'severity': Severity.TIDY,
'description': 'clang-tidy ' + description, 'description': 'clang-tidy ' + description,
'patterns': [r'.*: .+\[' + pattern + r'\]$'] 'patterns': patterns,
} }
def tidy_warn_pattern(description, pattern):
return tidy_warn(description, [r'.*: .+\[' + pattern + r'\]$'])
def simple_tidy_warn_pattern(description): def simple_tidy_warn_pattern(description):
return tidy_warn_pattern(description, description) return tidy_warn_pattern(description, description)
@@ -168,6 +172,8 @@ warn_patterns = [
simple_tidy_warn_pattern('portability-simd-intrinsics'), simple_tidy_warn_pattern('portability-simd-intrinsics'),
group_tidy_warn_pattern('portability'), group_tidy_warn_pattern('portability'),
tidy_warn('TIMEOUT', [r".*: warning: clang-tidy aborted "]),
# warnings from clang-tidy's clang-analyzer checks # warnings from clang-tidy's clang-analyzer checks
analyzer_high('clang-analyzer-core, null pointer', analyzer_high('clang-analyzer-core, null pointer',
[r".*: warning: .+ pointer is null .*\[clang-analyzer-core"]), [r".*: warning: .+ pointer is null .*\[clang-analyzer-core"]),