diff options
| author | Joe Perches | 2013-11-13 00:10:08 +0100 |
|---|---|---|
| committer | Linus Torvalds | 2013-11-13 04:09:24 +0100 |
| commit | 11ea516a6c578564a65a352abb08ef558d365946 (patch) | |
| tree | b9373f23d089a5c762ee72c52cd9c3e1d4e11222 /scripts/checkpatch.pl | |
| parent | checkpatch: update seq_<foo> tests (diff) | |
| download | kernel-qcow2-linux-11ea516a6c578564a65a352abb08ef558d365946.tar.gz kernel-qcow2-linux-11ea516a6c578564a65a352abb08ef558d365946.tar.xz kernel-qcow2-linux-11ea516a6c578564a65a352abb08ef558d365946.zip | |
checkpatch: find CamelCase definitions of struct/union/enum
Checkpatch doesn't currently find CamelCase definitions of structs, unions
or enums.
Add that ability.
Signed-off-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/checkpatch.pl')
| -rwxr-xr-x | scripts/checkpatch.pl | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl index 42567bcd66b2..fcc74fe7b4d9 100755 --- a/scripts/checkpatch.pl +++ b/scripts/checkpatch.pl @@ -443,8 +443,9 @@ sub seed_camelcase_file { next if ($line !~ /(?:[A-Z][a-z]|[a-z][A-Z])/); if ($line =~ /^[ \t]*(?:#[ \t]*define|typedef\s+$Type)\s+(\w*(?:[A-Z][a-z]|[a-z][A-Z])\w*)/) { $camelcase{$1} = 1; - } - elsif ($line =~ /^\s*$Declare\s+(\w*(?:[A-Z][a-z]|[a-z][A-Z])\w*)\s*[\(\[,;]/) { + } elsif ($line =~ /^\s*$Declare\s+(\w*(?:[A-Z][a-z]|[a-z][A-Z])\w*)\s*[\(\[,;]/) { + $camelcase{$1} = 1; + } elsif ($line =~ /^\s*(?:union|struct|enum)\s+(\w*(?:[A-Z][a-z]|[a-z][A-Z])\w*)\s*[;\{]/) { $camelcase{$1} = 1; } } |