diff options
| author | Andy Whitcroft <apw@shadowen.org> | 2008-10-15 22:02:33 -0700 |
|---|---|---|
| committer | Linus Torvalds <torvalds@linux-foundation.org> | 2008-10-16 11:21:37 -0700 |
| commit | bf30d6ede099c2a31b13bbd05b1022dc0da684d5 (patch) | |
| tree | 54bf58e8077d06b7430b08044241cc230d5f5fca /scripts | |
| parent | f16fa28f7b3d95e989fc64c8480e44c1bcf4bac3 (diff) | |
| download | op-kernel-dev-bf30d6ede099c2a31b13bbd05b1022dc0da684d5.zip op-kernel-dev-bf30d6ede099c2a31b13bbd05b1022dc0da684d5.tar.gz | |
checkpatch: complex macros checks miss square brackets
We are missing 'simple' values which include square brackets. Refactor to
ensure we handle nesting correctly and detect these simple forms.
Signed-off-by: Andy Whitcroft <apw@shadowen.org>
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 | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl index c479bde..54dfa2b 100755 --- a/scripts/checkpatch.pl +++ b/scripts/checkpatch.pl @@ -2142,9 +2142,10 @@ sub process { $dstat =~ s/\s*$//s; # Flatten any parentheses and braces - while ($dstat =~ s/\([^\(\)]*\)/1/) { - } - while ($dstat =~ s/\{[^\{\}]*\}/1/) { + while ($dstat =~ s/\([^\(\)]*\)/1/ || + $dstat =~ s/\{[^\{\}]*\}/1/ || + $dstat =~ s/\[[^\{\}]*\]/1/) + { } my $exceptions = qr{ |
