aboutsummaryrefslogtreecommitdiffstats
path: root/scripts
diff options
context:
space:
mode:
authorJoe Perches <joe@perches.com>2014-12-10 18:51:37 -0500
committerLinus Torvalds <torvalds@linux-foundation.org>2014-12-10 20:41:11 -0500
commit04941aa774320eb0118339c8cd11d7f1321e74d3 (patch)
tree18b686448c7595550bf5736032b2ab480f68dc7e /scripts
parent619a908aa334c854594e690fdbf1fe37b0e3e740 (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-xscripts/checkpatch.pl3
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) {