[U-Boot] [PATCH] checkpatch: warn of whitespace before semicolon at end of line.
Signed-off-by: Eric Nelson eric.nelson@boundarydevices.com --- tools/checkpatch.pl | 6 ++++++ 1 files changed, 6 insertions(+), 0 deletions(-)
diff --git a/tools/checkpatch.pl b/tools/checkpatch.pl index 2048a44..0caffcb 100755 --- a/tools/checkpatch.pl +++ b/tools/checkpatch.pl @@ -3139,6 +3139,12 @@ sub process { "Statements terminations use 1 semicolon\n" . $herecurr); }
+# check for multiple semicolons + if ($line =~ /\s+;$/) { + WARN("SPACEBEFORE_SEMICOLON", + "Blank before semicolon\n" . $herecurr); + } + # check for gcc specific __FUNCTION__ if ($line =~ /__FUNCTION__/) { WARN("USE_FUNC",
Dear Eric Nelson,
Signed-off-by: Eric Nelson eric.nelson@boundarydevices.com
tools/checkpatch.pl | 6 ++++++ 1 files changed, 6 insertions(+), 0 deletions(-)
diff --git a/tools/checkpatch.pl b/tools/checkpatch.pl index 2048a44..0caffcb 100755 --- a/tools/checkpatch.pl +++ b/tools/checkpatch.pl @@ -3139,6 +3139,12 @@ sub process { "Statements terminations use 1 semicolon\n" .
$herecurr);
}+# check for multiple semicolons
Are you sure about the comment ^ ;-)
if ($line =~ /\s+;$/) {WARN("SPACEBEFORE_SEMICOLON","Blank before semicolon\n" . $herecurr);}# check for gcc specific __FUNCTION__ if ($line =~ /__FUNCTION__/) { WARN("USE_FUNC",
Best regards, Marek Vasut
On 05/02/2012 08:00 PM, Mike Frysinger wrote:
On Tuesday 01 May 2012 18:06:16 Eric Nelson wrote:
tools/checkpatch.pl | 6 ++++++ 1 files changed, 6 insertions(+), 0 deletions(-)
should be sent to lkml/kbuild lists ... -mike
Thanks for the pointer Mike.
participants (3)
-
Eric Nelson -
Marek Vasut -
Mike Frysinger