From e8f4a716ad909ef7e92fccf503a8f3fc6d0c483f Mon Sep 17 00:00:00 2001 From: Chih-Hung Hsieh Date: Fri, 18 Sep 2020 21:51:06 -0700 Subject: Add new Java/C++/Yacc warning patterns Test: warn.py --url=http://cs/android --separator='?l=' build.log > warnings.html Test: warn.py --gencsv build.log > warnings.csv Change-Id: I49eafa322902f0587738b583b6262117595318ee --- tools/warn/cpp_warn_patterns.py | 11 +++++++++-- 1 file changed, 9 insertions(+), 2 deletions(-) (limited to 'tools/warn/cpp_warn_patterns.py') diff --git a/tools/warn/cpp_warn_patterns.py b/tools/warn/cpp_warn_patterns.py index 65ce73ac6b..e8783bc476 100644 --- a/tools/warn/cpp_warn_patterns.py +++ b/tools/warn/cpp_warn_patterns.py @@ -155,6 +155,7 @@ warn_patterns = [ [r".*: warning: unknown attribute '.+'"]), medium('Attribute ignored', [r".*: warning: '_*packed_*' attribute ignored", + r".*: warning: .* not supported .*Wignored-attributes", r".*: warning: attribute declaration must precede definition .+ignored-attributes"]), medium('Visibility problem', [r".*: warning: declaration of '.+' will not be visible outside of this function"]), @@ -251,6 +252,8 @@ warn_patterns = [ [r".*: warning: taking address of temporary"]), medium('Taking address of packed member', [r".*: warning: taking address of packed member"]), + medium('Pack alignment value is modified', + [r".*: warning: .*#pragma pack alignment value is modified.*Wpragma-pack.*"]), medium('Possible broken line continuation', [r".*: warning: backslash and newline separated by space"]), medium('Undefined variable template', @@ -332,7 +335,7 @@ warn_patterns = [ [r".*: warning: extra tokens at end of #endif directive"]), medium('Comparison between different enums', [r".*: warning: comparison between '.+' and '.+'.+Wenum-compare", - r".*: warning: comparison of .* enumeration types .*-Wenum-compare-switch"]), + r".*: warning: comparison of .* enumeration types .*-Wenum-compare.*"]), medium('Conversion may change value', [r".*: warning: converting negative value '.+' to '.+'", r".*: warning: conversion to '.+' .+ may (alter|change)"]), @@ -396,6 +399,8 @@ warn_patterns = [ r".*: warning: absolute value function '.+' given .+ which may cause truncation .+Wabsolute-value"]), low('Using C++11 extensions', [r".*: warning: 'auto' type specifier is a C\+\+11 extension"]), + low('Using C++17 extensions', + [r".*: warning: .* a C\+\+17 extension .+Wc\+\+17-extensions"]), low('Refers to implicitly defined namespace', [r".*: warning: using directive refers to implicitly-defined namespace .+"]), low('Invalid pp token', @@ -437,8 +442,10 @@ warn_patterns = [ [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', + medium('Overlapping comparisons', [r".*: warning: overlapping comparisons.+Wtautological-overlap-compare"]), + medium('bitwise comparison', + [r".*: warning: bitwise comparison.+Wtautological-bitwise-compare"]), medium('int in bool context', [r".*: warning: converting.+to a boolean.+Wint-in-bool-context"]), medium('bitwise conditional parentheses', -- cgit v1.2.3-59-g8ed1b