tequilaOS/platform_build
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Merge "Recognize more clang-tidy compiler warnings." am: 62231c8fd2

Browse source 
am: 7094388d83

Change-Id: I3188cb65d9c2181bc762105085630edf4df9639d
This commit is contained in:
Chih-Hung Hsieh 2018-06-04 16:32:22 -07:00 committed by android-build-merger
commit a34d1fc85f
1 changed files with 29 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

30
tools/warn.py
View file

@ -177,6 +177,9 @@ warn_patterns = [
{'category': 'make', 'severity': Severity.MEDIUM,
'description': 'Invalid SDK/NDK linking',
'patterns': [r".*: warning: .+ \(.+\) should not link to .+ \(.+\)"]},
{'category': 'make', 'severity': Severity.MEDIUM,
'description': 'Duplicate header copy',
'patterns': [r".*: warning: Duplicate header copy: .+"]},
{'category': 'C/C++', 'severity': Severity.HIGH, 'option': '-Wimplicit-function-declaration',
'description': 'Implicit function declaration',
'patterns': [r".*: warning: implicit declaration of function .+",
@ -238,9 +241,11 @@ warn_patterns = [
'description': 'Unused parameter',
'patterns': [r".*: warning: unused parameter '.*'"]},
{'category': 'C/C++', 'severity': Severity.MEDIUM, 'option': '-Wunused',
'description': 'Unused function, variable or label',
'description': 'Unused function, variable, label, comparison, etc.',
'patterns': [r".*: warning: '.+' defined but not used",
r".*: warning: unused function '.+'",
r".*: warning: unused label '.+'",
r".*: warning: relational comparison result unused",
r".*: warning: lambda capture .* is not used",
r".*: warning: private field '.+' is not used",
r".*: warning: unused variable '.+'"]},
@ -2513,6 +2518,29 @@ warn_patterns = [
# warnings from clang-tidy
group_tidy_warn_pattern('android'),
simple_tidy_warn_pattern('bugprone-argument-comment'),
simple_tidy_warn_pattern('bugprone-copy-constructor-init'),
simple_tidy_warn_pattern('bugprone-fold-init-type'),
simple_tidy_warn_pattern('bugprone-forward-declaration-namespace'),
simple_tidy_warn_pattern('bugprone-forwarding-reference-overload'),
simple_tidy_warn_pattern('bugprone-inaccurate-erase'),
simple_tidy_warn_pattern('bugprone-incorrect-roundings'),
simple_tidy_warn_pattern('bugprone-integer-division'),
simple_tidy_warn_pattern('bugprone-lambda-function-name'),
simple_tidy_warn_pattern('bugprone-macro-parentheses'),
simple_tidy_warn_pattern('bugprone-misplaced-widening-cast'),
simple_tidy_warn_pattern('bugprone-move-forwarding-reference'),
simple_tidy_warn_pattern('bugprone-sizeof-expression'),
simple_tidy_warn_pattern('bugprone-string-constructor'),
simple_tidy_warn_pattern('bugprone-string-integer-assignment'),
simple_tidy_warn_pattern('bugprone-suspicious-enum-usage'),
simple_tidy_warn_pattern('bugprone-suspicious-missing-comma'),
simple_tidy_warn_pattern('bugprone-suspicious-string-compare'),
simple_tidy_warn_pattern('bugprone-suspicious-semicolon'),
simple_tidy_warn_pattern('bugprone-undefined-memory-manipulation'),
simple_tidy_warn_pattern('bugprone-unused-raii'),
simple_tidy_warn_pattern('bugprone-use-after-move'),
group_tidy_warn_pattern('bugprone'),
group_tidy_warn_pattern('cert'),
group_tidy_warn_pattern('clang-diagnostic'),
group_tidy_warn_pattern('cppcoreguidelines'),