diff options
author | Joe Perches <joe@perches.com> | 2016-08-02 17:04:42 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2016-08-02 19:35:12 -0400 |
commit | c844711575086231890084390a275d06f11a623a (patch) | |
tree | 3a8fabb3a2b4a364dd03fa4d1b040f1464c23daa | |
parent | cec3aaa56638c7aad763630b9cbe591f2e791a3b (diff) |
checkpatch: improve 'bare use of' signed/unsigned types warning
Fix false positive warning of identifiers ending in signed with an =
assignment of WARNING: Prefer 'signed int' to bare use of 'signed'.
Link: http://lkml.kernel.org/r/6a0e24c3e9102337528ecfcbbe91a0eb5b4820ed.1469529497.git.joe@perches.com
Signed-off-by: Joe Perches <joe@perches.com>
Reported-by: Alan Douglas <alanjhd@gmail.com>
Acked-by: Andy Whitcroft <apw@canonical.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
-rwxr-xr-x | scripts/checkpatch.pl | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl index 77915e095022..1d5b09dd577a 100755 --- a/scripts/checkpatch.pl +++ b/scripts/checkpatch.pl | |||
@@ -3347,7 +3347,7 @@ sub process { | |||
3347 | next if ($line =~ /^[^\+]/); | 3347 | next if ($line =~ /^[^\+]/); |
3348 | 3348 | ||
3349 | # check for declarations of signed or unsigned without int | 3349 | # check for declarations of signed or unsigned without int |
3350 | while ($line =~ m{($Declare)\s*(?!char\b|short\b|int\b|long\b)\s*($Ident)?\s*[=,;\[\)\(]}g) { | 3350 | while ($line =~ m{\b($Declare)\s*(?!char\b|short\b|int\b|long\b)\s*($Ident)?\s*[=,;\[\)\(]}g) { |
3351 | my $type = $1; | 3351 | my $type = $1; |
3352 | my $var = $2; | 3352 | my $var = $2; |
3353 | $var = "" if (!defined $var); | 3353 | $var = "" if (!defined $var); |