diff options
| author | Treehugger Robot <treehugger-gerrit@google.com> | 2018-06-04 20:20:15 +0000 |
|---|---|---|
| committer | Gerrit Code Review <noreply-gerritcodereview@google.com> | 2018-06-04 20:20:15 +0000 |
| commit | 62231c8fd246832d198428194f386671f4d36328 (patch) | |
| tree | 2c4c02d91bfb7d62745144f1c23de17c90edb4f7 | |
| parent | 89b40fa6455cffe89d3db4886c745ff327191f74 (diff) | |
| parent | 9e9355dac603e0d0105c7d8500a064e8b2ca4f65 (diff) | |
| download | build-android-o-mr1-iot-release-1.0.1.tar.gz | |
Merge "Recognize more clang-tidy compiler warnings."android-o-mr1-iot-release-1.0.1
| -rwxr-xr-x | tools/warn.py | 30 |
1 files changed, 29 insertions, 1 deletions
diff --git a/tools/warn.py b/tools/warn.py index 01398be550..efa064f1c4 100755 --- a/tools/warn.py +++ b/tools/warn.py @@ -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'), |