Messages in this thread Patch in this message | | | From | Cody P Schafer <> | Subject | [PATCH v2] checkpatch: avoid warning on page flag acessors | Date | Tue, 14 May 2013 10:06:51 -0700 |
| |
Using Page*() triggers a camelcase warning, but shouldn't.
be987d9f80354e2e919926349282facd74992f90 added a spurious '"' (double quote) breaking the regex.
Signed-off-by: Cody P Schafer <cody@linux.vnet.ibm.com> --- scripts/checkpatch.pl | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl index b954de5..ee24026 100755 --- a/scripts/checkpatch.pl +++ b/scripts/checkpatch.pl @@ -2939,7 +2939,7 @@ sub process { my $var = $1; if ($var !~ /$Constant/ && $var =~ /[A-Z]\w*[a-z]|[a-z]\w*[A-Z]/ && - $var !~ /"^(?:Clear|Set|TestClear|TestSet|)Page[A-Z]/ && + $var !~ /"^(?:Clear|Set|TestClear|TestSet)?Page[A-Z]/ && !defined $camelcase{$var}) { $camelcase{$var} = 1; WARN("CAMELCASE", -- 1.8.2.2
| |