diff options
author | Andy Whitcroft <apw@canonical.com> | 2009-02-27 17:03:08 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2009-02-27 19:26:22 -0500 |
commit | 9360b0e50e9f3e0fd70a077b4ede9885ebc21720 (patch) | |
tree | 065fa2a65de26e3a720a043b7eb699dc66dd7c42 /scripts | |
parent | 667026e7b082ad59eb7194d6b5d159ed6c340e05 (diff) |
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')
-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 { | |||
1584 | } | 1584 | } |
1585 | # TEST: allow direct testing of the attribute matcher. | 1585 | # TEST: allow direct testing of the attribute matcher. |
1586 | if ($dbg_attr) { | 1586 | if ($dbg_attr) { |
1587 | if ($line =~ /^.\s*$Attribute\s*$/) { | 1587 | if ($line =~ /^.\s*$Modifier\s*$/) { |
1588 | ERROR("TEST: is attr\n" . $herecurr); | 1588 | ERROR("TEST: is attr\n" . $herecurr); |
1589 | } elsif ($dbg_attr > 1 && $line =~ /^.+($Attribute)/) { | 1589 | } elsif ($dbg_attr > 1 && $line =~ /^.+($Modifier)/) { |
1590 | ERROR("TEST: is not attr ($1 is)\n". $herecurr); | 1590 | ERROR("TEST: is not attr ($1 is)\n". $herecurr); |
1591 | } | 1591 | } |
1592 | next; | 1592 | next; |