Messages in this thread Patch in this message | | | Date | Thu, 16 Aug 2007 13:22:15 +0800 | From | Eugene Teo <> | Subject | [PATCH] Make checkpatch rant about trailing ; at the end of "if" expr |
| |
Make checkpatch rant about trailing ; at the end of "if" expression.
Thanks to Auke for the regexp.
Signed-off by: Eugene Teo <eugeneteo@kernel.sg>
--- checkpatch.pl-0.09.default 2007-08-03 23:31:40.000000000 +0800 +++ checkpatch.pl-0.09 2007-08-16 13:18:40.000000000 +0800 @@ -1091,6 +1091,12 @@ sub process { CHK("__setup appears un-documented -- check Documentation/kernel-parameters.txt\n" . $herecurr); } } + +# checks for trailing ; at the end of "if" expression + if ($line =~ /\bif\s*\([^\)]*\)\s*\;/) { + my $herevet = "$here\n" . cat_vet($line) . "\n"; + ERROR("trailing ;\n" . $herevet); + } } if ($chk_patch && !$is_patch) { - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |