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

Commit 4fc3ef6c authored by Chih-hung Hsieh's avatar Chih-hung Hsieh Committed by Automerger Merge Worker
Browse files

Merge "Add new clang-tidy and clang warning patterns" am: 95d9ba93 

Original change: https://android-review.googlesource.com/c/platform/build/+/1850842

Change-Id: Id33fb9e4bcf6e50e4e46c62d52af7d14590b3064
parents b808b63b 95d9ba93

tools/warn/cpp_warn_patterns.py

+2 −0
Original line number Diff line number Diff line
@@ -297,6 +297,7 @@ warn_patterns = [ 
           [r".*: warning: declaration 'class .+' does not declare anything"]),

    medium('Initialization order will be different',

           [r".*: warning: '.+' will be initialized after",

            r".*: warning: initializer order does not match the declaration order",

            r".*: warning: field .+ will be initialized after .+Wreorder"]),

    skip('skip,   ....',

         [r".*: warning:   '.+'"]),
@@ -448,6 +449,7 @@ warn_patterns = [ 
           [r".*: warning: 'operator new' must not return NULL unless it is declared 'throw\(\)' .+"]),

    medium('NULL used in arithmetic',

           [r".*: warning: NULL used in arithmetic",

            r".*: warning: .* subtraction with a null pointer",

            r".*: warning: comparison between NULL and non-pointer"]),

    medium('Misspelled header guard',

           [r".*: warning: '.+' is used as a header guard .+ followed by .+ different macro"]),

tools/warn/tidy_warn_patterns.py

+7 −2
Original line number Diff line number Diff line
@@ -81,6 +81,7 @@ def analyzer_group_check(check): 
warn_patterns = [

    # pylint does not recognize g-inconsistent-quotes

    # pylint:disable=line-too-long,bad-option-value,g-inconsistent-quotes

    group_tidy_warn_pattern('altera'),

    group_tidy_warn_pattern('android'),

    simple_tidy_warn_pattern('abseil-string-find-startswith'),

    simple_tidy_warn_pattern('bugprone-argument-comment'),
@@ -127,8 +128,9 @@ warn_patterns = [ 
    simple_tidy_warn_pattern('cert-oop54-cpp'),

    group_tidy_warn_pattern('cert'),

    group_tidy_warn_pattern('clang-diagnostic'),

    group_tidy_warn_pattern('concurrency'),

    group_tidy_warn_pattern('cppcoreguidelines'),

    group_tidy_warn_pattern('llvm'),

    group_tidy_warn_pattern('fuchsia'),

    simple_tidy_warn_pattern('google-default-arguments'),

    simple_tidy_warn_pattern('google-runtime-int'),

    simple_tidy_warn_pattern('google-runtime-operator'),
@@ -152,8 +154,10 @@ warn_patterns = [ 
    simple_tidy_warn_pattern('hicpp-noexcept-move'),

    simple_tidy_warn_pattern('hicpp-use-override'),

    group_tidy_warn_pattern('hicpp'),

    group_tidy_warn_pattern('modernize'),

    group_tidy_warn_pattern('llvm'),

    group_tidy_warn_pattern('llvmlibc'),

    group_tidy_warn_pattern('misc'),

    group_tidy_warn_pattern('modernize'),

    simple_tidy_warn_pattern('performance-faster-string-find'),

    simple_tidy_warn_pattern('performance-for-range-copy'),

    simple_tidy_warn_pattern('performance-implicit-cast-in-loop'),
@@ -173,6 +177,7 @@ warn_patterns = [ 
    group_tidy_warn_pattern('portability'),



    tidy_warn('TIMEOUT', [r".*: warning: clang-tidy aborted "]),

    tidy_warn('Long Runs', [r".*: warning: clang-tidy used "]),



    # warnings from clang-tidy's clang-analyzer checks

    analyzer_high('clang-analyzer-core, null pointer',