summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorEric Nelson <eric.nelson@boundarydevices.com>2012-05-02 20:32:18 -0700
committerWolfgang Denk <wd@denx.de>2012-06-21 21:30:02 +0200
commit172a3a82afd042b8cf43a0cd9720195985a46e3c (patch)
tree3e727c550b32136a58d1a1aecbdce66e718a8511
parentb37d41aa24aedd84f718d019e5b7a27a06aa73fc (diff)
downloadu-boot-imx-172a3a82afd042b8cf43a0cd9720195985a46e3c.zip
u-boot-imx-172a3a82afd042b8cf43a0cd9720195985a46e3c.tar.gz
u-boot-imx-172a3a82afd042b8cf43a0cd9720195985a46e3c.tar.bz2
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-xtools/checkpatch.pl6
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",