diff options
author | Jean Delvare <jdelvare@suse.de> | 2017-09-08 19:16:11 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2017-09-08 21:26:50 -0400 |
commit | 0547fa5851c921eb1f64286f6b2ae7ab5df24e6e (patch) | |
tree | adef0edadb68edd1191e9d917f10e413560bc681 /scripts | |
parent | 0675a8fbd71c1a0b605ce82adea37131452f1070 (diff) |
checkpatch: add 6 missing types to --list-types
Unlike all other types, LONG_LINE, LONG_LINE_COMMENT and LONG_LINE_STRING
are passed to WARN() through a variable. This causes the parser in
list_types() to miss them and consequently they are not present in the
output of --list-types.
Additionally, types TYPO_SPELLING, FSF_MAILING_ADDRESS and AVOID_BUG are
passed with a variable level, causing the parser to miss them too.
So modify the regex to also catch these special cases.
Link: http://lkml.kernel.org/r/20170902175610.7e4a7c9d@endymion
Fixes: 3beb42eced39 ("checkpatch: add --list-types to show message types to show or ignore")
Signed-off-by: Jean Delvare <jdelvare@suse.de>
Acked-by: Joe Perches <joe@perches.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'scripts')
-rwxr-xr-x | scripts/checkpatch.pl | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl index b3a1bc70d9ee..dd2c262aebbf 100755 --- a/scripts/checkpatch.pl +++ b/scripts/checkpatch.pl | |||
@@ -145,7 +145,8 @@ sub list_types { | |||
145 | close($script); | 145 | close($script); |
146 | 146 | ||
147 | my @types = (); | 147 | my @types = (); |
148 | for ($text =~ /\b(?:(?:CHK|WARN|ERROR)\s*\(\s*"([^"]+)")/g) { | 148 | # Also catch when type or level is passed through a variable |
149 | for ($text =~ /(?:(?:\bCHK|\bWARN|\bERROR|&\{\$msg_level})\s*\(|\$msg_type\s*=)\s*"([^"]+)"/g) { | ||
149 | push (@types, $_); | 150 | push (@types, $_); |
150 | } | 151 | } |
151 | @types = sort(uniq(@types)); | 152 | @types = sort(uniq(@types)); |