Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 7094388d authored by Chih-Hung Hsieh's avatar Chih-Hung Hsieh Committed by android-build-merger
Browse files

Merge "Recognize more clang-tidy compiler warnings." 

am: 62231c8f

Change-Id: Ie3d5807683e0125ae0d88479096e3d074b4c90f9
parents 0b5ec446 62231c8f

tools/warn.py

+29 −1
Original line number Original line Diff line number Diff line
@@ -177,6 +177,9 @@ warn_patterns = [ 
    {'category': 'make', 'severity': Severity.MEDIUM,

    {'category': 'make', 'severity': Severity.MEDIUM,

     'description': 'Invalid SDK/NDK linking',

     'description': 'Invalid SDK/NDK linking',

     'patterns': [r".*: warning: .+ \(.+\) should not link to .+ \(.+\)"]},

     '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',

    {'category': 'C/C++', 'severity': Severity.HIGH, 'option': '-Wimplicit-function-declaration',

     'description': 'Implicit function declaration',

     'description': 'Implicit function declaration',

     'patterns': [r".*: warning: implicit declaration of function .+",

     'patterns': [r".*: warning: implicit declaration of function .+",
@@ -238,9 +241,11 @@ warn_patterns = [ 
     'description': 'Unused parameter',

     'description': 'Unused parameter',

     'patterns': [r".*: warning: unused parameter '.*'"]},

     'patterns': [r".*: warning: unused parameter '.*'"]},

    {'category': 'C/C++', 'severity': Severity.MEDIUM, 'option': '-Wunused',

    {'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",

     'patterns': [r".*: warning: '.+' defined but not used",

                  r".*: warning: unused function '.+'",

                  r".*: warning: unused function '.+'",

                  r".*: warning: unused label '.+'",

                  r".*: warning: relational comparison result unused",

                  r".*: warning: lambda capture .* is not used",

                  r".*: warning: lambda capture .* is not used",

                  r".*: warning: private field '.+' is not used",

                  r".*: warning: private field '.+' is not used",

                  r".*: warning: unused variable '.+'"]},

                  r".*: warning: unused variable '.+'"]},
@@ -2513,6 +2518,29 @@ warn_patterns = [ 




    # warnings from clang-tidy

    # warnings from clang-tidy

    group_tidy_warn_pattern('android'),

    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('cert'),

    group_tidy_warn_pattern('clang-diagnostic'),

    group_tidy_warn_pattern('clang-diagnostic'),

    group_tidy_warn_pattern('cppcoreguidelines'),

    group_tidy_warn_pattern('cppcoreguidelines'),