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

Commit 01530a6f authored by Chih-Hung Hsieh's avatar Chih-Hung Hsieh Committed by Chih-hung Hsieh
Browse files

Recognize new warning messages. 

* Generalize regular expression to match new warning messages.
* Remove old unused pattern.

Change-Id: I54f2e5bdb255b290cb3751ba56558f5fe8d7245e
Test: Run warn.py with build.log
parent ce4c49c2

tools/warn.py

+1 −4
Original line number Diff line number Diff line
@@ -1508,7 +1508,7 @@ warnpatterns = [ 
    { 'category':'C/C++',   'severity':severity.MEDIUM,   'members':[], 'option':'-Wconversion',

        'description':'Conversion may change value',

        'patterns':[r".*: warning: converting negative value '.+' to '.+'",

                    r".*: warning: conversion to '.+' .+ may alter its value"] },

                    r".*: warning: conversion to '.+' .+ may (alter|change)"] },

    { 'category':'C/C++',   'severity':severity.MEDIUM,   'members':[], 'option':'-Wconversion-null',

        'description':'Converting to non-pointer type from NULL',

        'patterns':[r".*: warning: converting to non-pointer type '.+' from NULL"] },
@@ -1551,9 +1551,6 @@ warnpatterns = [ 
    { 'category':'C/C++',   'severity':severity.HIGH,     'members':[], 'option':'-Wextra',

        'description':'Base should be explicitly initialized in copy constructor',

        'patterns':[r".*: warning: base class '.+' should be explicitly initialized in the copy constructor"] },

    { 'category':'C/C++',   'severity':severity.MEDIUM,     'members':[], 'option':'',

        'description':'Converting from <type> to <other type>',

        'patterns':[r".*: warning: converting to '.+' from '.+'"] },

    { 'category':'C/C++',   'severity':severity.MEDIUM,     'members':[], 'option':'',

        'description':'VLA has zero or negative size',

        'patterns':[r".*: warning: Declared variable-length array \(VLA\) has .+ size"] },