* [U-Boot] [PATCH V2] checkpatch: warn of whitespace before semicolon at end of line. [not found] <http://lists.denx.de/pipermail/u-boot/2012-May#123595> @ 2012-05-01 23:06 ` Eric Nelson 2012-05-01 23:11 ` Marek Vasut 0 siblings, 1 reply; 3+ messages in thread From: Eric Nelson @ 2012-05-01 23:06 UTC (permalink / raw) To: u-boot 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 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", -- 1.7.9 ^ permalink raw reply related [flat|nested] 3+ messages in thread
* [U-Boot] [PATCH V2] checkpatch: warn of whitespace before semicolon at end of line. 2012-05-01 23:06 ` [U-Boot] [PATCH V2] checkpatch: warn of whitespace before semicolon at end of line Eric Nelson @ 2012-05-01 23:11 ` Marek Vasut 2012-05-01 23:32 ` Eric Nelson 0 siblings, 1 reply; 3+ messages in thread From: Marek Vasut @ 2012-05-01 23:11 UTC (permalink / raw) To: u-boot 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 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 <marex@denx.de> Best regards, Marek Vasut ^ permalink raw reply [flat|nested] 3+ messages in thread
* [U-Boot] [PATCH V2] checkpatch: warn of whitespace before semicolon at end of line. 2012-05-01 23:11 ` Marek Vasut @ 2012-05-01 23:32 ` Eric Nelson 0 siblings, 0 replies; 3+ messages in thread From: Eric Nelson @ 2012-05-01 23:32 UTC (permalink / raw) To: u-boot On 05/01/2012 04:11 PM, Marek Vasut wrote: > 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 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 :) > That looks to be Andy. It's not clear to me that this is a style violation though: http://www.kernel.org/doc/Documentation/CodingStyle Andy, we're discussing the inclusion of a test in checkpatch for space before semicolon at the end-of line to catch my bad habits like this: return 0 ; > For me, add > Acked-by: Marek Vasut<marex@denx.de> > ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2012-05-01 23:32 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <http://lists.denx.de/pipermail/u-boot/2012-May#123595>
2012-05-01 23:06 ` [U-Boot] [PATCH V2] checkpatch: warn of whitespace before semicolon at end of line Eric Nelson
2012-05-01 23:11 ` Marek Vasut
2012-05-01 23:32 ` Eric Nelson
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox