diff options
author | Eric Nelson <eric.nelson@boundarydevices.com> | 2012-05-03 03:32:18 (GMT) |
---|---|---|
committer | Wolfgang Denk <wd@denx.de> | 2012-06-21 19:30:02 (GMT) |
commit | 172a3a82afd042b8cf43a0cd9720195985a46e3c (patch) | |
tree | 3e727c550b32136a58d1a1aecbdce66e718a8511 | |
parent | b37d41aa24aedd84f718d019e5b7a27a06aa73fc (diff) | |
download | u-boot-172a3a82afd042b8cf43a0cd9720195985a46e3c.tar.xz |
checkpatch: add check for whitespace before semicolon at end-of-line
This tests for a bad habits of mine like this:
return 0 ;
Signed-off-by: Eric Nelson <eric.nelson@boundarydevices.com>
-rwxr-xr-x | tools/checkpatch.pl | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/tools/checkpatch.pl b/tools/checkpatch.pl index 2048a44..c7475f9 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", + "Whitespace before semicolon\n" . $herecurr); + } + # check for gcc specific __FUNCTION__ if ($line =~ /__FUNCTION__/) { WARN("USE_FUNC", |