From mboxrd@z Thu Jan 1 00:00:00 1970 From: Marek Vasut Date: Wed, 2 May 2012 01:11:25 +0200 Subject: [U-Boot] [PATCH V2] checkpatch: warn of whitespace before semicolon at end of line. In-Reply-To: <1335913584-13221-1-git-send-email-eric.nelson@boundarydevices.com> References: <1335913584-13221-1-git-send-email-eric.nelson@boundarydevices.com> Message-ID: <201205020111.26113.marex@denx.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Dear Eric Nelson, > Signed-off-by: Eric Nelson > --- > 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 whitespace before semicolon (not allowed at end-of-line) > + if ($line =~ /\s+;$/) { > + WARN("SPACEBEFORE_SEMICOLON", > + "Blank before semicolon\n" . $herecurr); > + } > + > # check for gcc specific __FUNCTION__ > if ($line =~ /__FUNCTION__/) { > WARN("USE_FUNC", Coolness, now let's also CC the linux (upstream) checkpatch author, to get it properly upstreamed :) For me, add Acked-by: Marek Vasut Best regards, Marek Vasut