diff options
author | Joe Perches <joe@perches.com> | 2014-12-10 18:51:37 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2014-12-10 20:41:11 -0500 |
commit | 04941aa774320eb0118339c8cd11d7f1321e74d3 (patch) | |
tree | 18b686448c7595550bf5736032b2ab480f68dc7e /scripts | |
parent | 619a908aa334c854594e690fdbf1fe37b0e3e740 (diff) |
checkpatch: improve test for no space after cast
sizeof(foo) is not a cast, allow a space after it.
Signed-off-by: Joe Perches <joe@perches.com>
Tested-by: Kalle Valo <kvalo@qca.qualcomm.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 | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl index 8a577aa03bc6..d94f5d879fb1 100755 --- a/scripts/checkpatch.pl +++ b/scripts/checkpatch.pl | |||
@@ -2515,7 +2515,8 @@ sub process { | |||
2515 | } | 2515 | } |
2516 | } | 2516 | } |
2517 | 2517 | ||
2518 | if ($line =~ /^\+.*\(\s*$Type\s*\)[ \t]+(?!$Assignment|$Arithmetic|{)/) { | 2518 | if ($line =~ /^\+.*(\w+\s*)?\(\s*$Type\s*\)[ \t]+(?!$Assignment|$Arithmetic|[,;\({\[\<\>])/ && |
2519 | (!defined($1) || $1 !~ /sizeof\s*/)) { | ||
2519 | if (CHK("SPACING", | 2520 | if (CHK("SPACING", |
2520 | "No space is necessary after a cast\n" . $herecurr) && | 2521 | "No space is necessary after a cast\n" . $herecurr) && |
2521 | $fix) { | 2522 | $fix) { |