diff options
| author |   Andy Whitcroft <apw@canonical.com> | 2009-02-27 14:03:08 -0800 |
|---|---|---|
| committer |   Linus Torvalds <torvalds@linux-foundation.org> | 2009-02-27 16:26:22 -0800 |
| commit | 9360b0e50e9f3e0fd70a077b4ede9885ebc21720 (patch) | |
| tree | 065fa2a65de26e3a720a043b7eb699dc66dd7c42 /scripts/checkpatch.pl | |
| parent | checkpatch: a modifier is not an identifier at the end of a type (diff) | |
| download | linux-dev-9360b0e50e9f3e0fd70a077b4ede9885ebc21720.tar.xz linux-dev-9360b0e50e9f3e0fd70a077b4ede9885ebc21720.zip | |
checkpatch: extend attribute testing to all modifiers
We should allow testing of all modifiers not just attributes. Extend
testing and test for all the know modifiers.
Signed-off-by: Andy Whitcroft <apw@canonical.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'scripts/checkpatch.pl')
| -rwxr-xr-x | scripts/checkpatch.pl | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl index b9e9ee1315b7..0dcfdce632b1 100755 --- a/scripts/checkpatch.pl +++ b/scripts/checkpatch.pl @@ -1584,9 +1584,9 @@ sub process { } # TEST: allow direct testing of the attribute matcher. if ($dbg_attr) { - if ($line =~ /^.\s*$Attribute\s*$/) { + if ($line =~ /^.\s*$Modifier\s*$/) { ERROR("TEST: is attr\n" . $herecurr); - } elsif ($dbg_attr > 1 && $line =~ /^.+($Attribute)/) { + } elsif ($dbg_attr > 1 && $line =~ /^.+($Modifier)/) { ERROR("TEST: is not attr ($1 is)\n". $herecurr); } next; |