diff options
| -rw-r--r-- | tools/warn/cpp_warn_patterns.py | 24 | ||||
| -rw-r--r-- | tools/warn/java_warn_patterns.py | 11 |
2 files changed, 35 insertions, 0 deletions
diff --git a/tools/warn/cpp_warn_patterns.py b/tools/warn/cpp_warn_patterns.py index 0c458fb5b7..65ce73ac6b 100644 --- a/tools/warn/cpp_warn_patterns.py +++ b/tools/warn/cpp_warn_patterns.py @@ -435,6 +435,30 @@ warn_patterns = [ [r".*: warning: template argument.+Wunnamed-type-template-args"]), medium('Unannotated fall-through between switch labels', [r".*: warning: unannotated fall-through between switch labels.+Wimplicit-fallthrough"]), + medium('Invalid partial specialization', + [r".*: warning: class template partial specialization.+Winvalid-partial-specialization"]), + medium('Overlapping compatisons', + [r".*: warning: overlapping comparisons.+Wtautological-overlap-compare"]), + medium('int in bool context', + [r".*: warning: converting.+to a boolean.+Wint-in-bool-context"]), + medium('bitwise conditional parentheses', + [r".*: warning: operator.+has lower precedence.+Wbitwise-conditional-parentheses"]), + medium('sizeof array div', + [r".*: warning: .+number of elements in.+array.+Wsizeof-array-div"]), + medium('bool operation', + [r".*: warning: .+boolean.+always.+Wbool-operation"]), + medium('Undefined bool conversion', + [r".*: warning: .+may be.+always.+true.+Wundefined-bool-conversion"]), + medium('Typedef requires a name', + [r".*: warning: typedef requires a name.+Wmissing-declaration"]), + medium('Unknown escape sequence', + [r".*: warning: unknown escape sequence.+Wunknown-escape-sequence"]), + medium('Unicode whitespace', + [r".*: warning: treating Unicode.+as whitespace.+Wunicode-whitespace"]), + medium('Unused local typedef', + [r".*: warning: unused typedef.+Wunused-local-typedef"]), + medium('varargs warnings', + [r".*: warning: .*argument to 'va_start'.+\[-Wvarargs\]"]), harmless('Discarded qualifier from pointer target type', [r".*: warning: .+ discards '.+' qualifier from pointer target type"]), harmless('Use snprintf instead of sprintf', diff --git a/tools/warn/java_warn_patterns.py b/tools/warn/java_warn_patterns.py index 0a443d4835..96510b4e56 100644 --- a/tools/warn/java_warn_patterns.py +++ b/tools/warn/java_warn_patterns.py @@ -38,6 +38,10 @@ def java_medium(description, pattern_list): return java_warn(Severity.MEDIUM, description, pattern_list) +def java_medium_type(name): + return java_medium(name, [r'.*\.java:.*: warning: .+ \[' + name + r'\]$']) + + def java_low(description, pattern_list): return java_warn(Severity.LOW, description, pattern_list) @@ -457,6 +461,13 @@ warn_patterns = [ [r".*: warning: \[WaitNotInLoop\] .+"]), java_medium('A wakelock acquired with a timeout may be released by the system before calling `release`, even after checking `isHeld()`. If so, it will throw a RuntimeException. Please wrap in a try/catch block.', [r".*: warning: \[WakelockReleasedDangerously\] .+"]), + java_medium_type('CallbackName'), + java_medium_type('ExecutorRegistration'), + java_medium_type('ListenerLast'), + java_medium_type('MissingBuildMethod'), + java_medium_type('NoByteOrShort'), + java_medium_type('SetterReturnsThis'), + java_medium_type('UseIcu'), java_high('AndroidInjection.inject() should always be invoked before calling super.lifecycleMethod()', [r".*: warning: \[AndroidInjectionBeforeSuper\] .+"]), java_high('Use of class, field, or method that is not compatible with legacy Android devices', |