From 01530a6ff238d6526e6900dd5e69bfb4dcda10dc Mon Sep 17 00:00:00 2001 From: Chih-Hung Hsieh Date: Wed, 24 Aug 2016 12:24:32 -0700 Subject: [PATCH] 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 --- tools/warn.py | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/tools/warn.py b/tools/warn.py index f118263be..3a8ae664f 100755 --- a/tools/warn.py +++ b/tools/warn.py @@ -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"] }, @@ -1552,9 +1552,6 @@ warnpatterns = [ '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 to ', - '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"] }, { 'category':'C/C++', 'severity':severity.MEDIUM, 'members':[], 'option':'', -- 2.11.0