aboutsummaryrefslogtreecommitdiffstats
path: root/scripts
diff options
context:
space:
mode:
authorMichal Zylowski <michal.zylowski@intel.com>2018-08-22 00:58:08 -0400
committerLinus Torvalds <torvalds@linux-foundation.org>2018-08-22 13:52:49 -0400
commit6ad724e2a48fc24dd9788490d85a3490cb0117c1 (patch)
tree01ad0efaeb18b10fae8e0af5df84d2acd0d70d2b /scripts
parent56294112791ac11c6fcdc6544f1381f9272202db (diff)
checkpatch: check for space after "else" keyword
Current checkpatch implementation permits notation like } else{ in kernel code. It looks like oversight and inconsistency in checkpatch rules (e.g. instruction like 'do' is tested). Add regex for checking space after 'else' keyword and trigger error if space is not present. Link: http://lkml.kernel.org/r/1533545753-8870-1-git-send-email-michal.zylowski@intel.com Signed-off-by: Michal Zylowski <michal.zylowski@intel.com> Acked-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')
-rwxr-xr-xscripts/checkpatch.pl4
1 files changed, 2 insertions, 2 deletions
diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl
index eced9b303995..f43d95a25712 100755
--- a/scripts/checkpatch.pl
+++ b/scripts/checkpatch.pl
@@ -4531,11 +4531,11 @@ sub process {
4531 4531
4532#need space before brace following if, while, etc 4532#need space before brace following if, while, etc
4533 if (($line =~ /\(.*\)\{/ && $line !~ /\($Type\)\{/) || 4533 if (($line =~ /\(.*\)\{/ && $line !~ /\($Type\)\{/) ||
4534 $line =~ /do\{/) { 4534 $line =~ /\b(?:else|do)\{/) {
4535 if (ERROR("SPACING", 4535 if (ERROR("SPACING",
4536 "space required before the open brace '{'\n" . $herecurr) && 4536 "space required before the open brace '{'\n" . $herecurr) &&
4537 $fix) { 4537 $fix) {
4538 $fixed[$fixlinenr] =~ s/^(\+.*(?:do|\)))\{/$1 {/; 4538 $fixed[$fixlinenr] =~ s/^(\+.*(?:do|else|\)))\{/$1 {/;
4539 } 4539 }
4540 } 4540 }
4541 4541