diff options
| author | Eric Nelson <eric.nelson@boundarydevices.com> | 2012-05-02 20:32:18 -0700 |
|---|---|---|
| committer | Wolfgang Denk <wd@denx.de> | 2012-06-21 21:30:02 +0200 |
| commit | 172a3a82afd042b8cf43a0cd9720195985a46e3c (patch) | |
| tree | 3e727c550b32136a58d1a1aecbdce66e718a8511 /tools | |
| parent | b37d41aa24aedd84f718d019e5b7a27a06aa73fc (diff) | |
| download | blackbird-obmc-uboot-172a3a82afd042b8cf43a0cd9720195985a46e3c.tar.gz blackbird-obmc-uboot-172a3a82afd042b8cf43a0cd9720195985a46e3c.zip | |
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>
Diffstat (limited to 'tools')
| -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 2048a449b5..c7475f9e33 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", |
