aboutsummaryrefslogtreecommitdiffstats
path: root/scripts/checkpatch.pl
diff options
context:
space:
mode:
authorAndy Whitcroft <apw@canonical.com>2010-10-26 17:23:14 -0400
committerLinus Torvalds <torvalds@linux-foundation.org>2010-10-26 19:52:20 -0400
commit53a3c4487a05b8f26ef72fe434a750a3402c998f (patch)
treedfe0b1f85cbf1082ceb52b71bb157cc715e22293 /scripts/checkpatch.pl
parent9446ef569c288e683225fec8337a0b2b81e75cc5 (diff)
checkpatch: returning errno typically should be negative
Add a (strict mode only) test to check for non-negative returns of what appear to be errno values as the majority case these should indeed be negative. Suggested-by: Andrew Morton <akpm@linux-foundation.org> Signed-off-by: Andy Whitcroft <apw@canonical.com> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'scripts/checkpatch.pl')
-rwxr-xr-xscripts/checkpatch.pl7
1 files changed, 7 insertions, 0 deletions
diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl
index 3cec2990d51e..bcdb54bd61a0 100755
--- a/scripts/checkpatch.pl
+++ b/scripts/checkpatch.pl
@@ -2204,6 +2204,13 @@ sub process {
2204 ERROR("space required before the open parenthesis '('\n" . $herecurr); 2204 ERROR("space required before the open parenthesis '('\n" . $herecurr);
2205 } 2205 }
2206 } 2206 }
2207# Return of what appears to be an errno should normally be -'ve
2208 if ($line =~ /^.\s*return\s*(E[A-Z]*)\s*;/) {
2209 my $name = $1;
2210 if ($name ne 'EOF' && $name ne 'ERROR') {
2211 WARN("return of an errno should typically be -ve (return -$1)\n" . $herecurr);
2212 }
2213 }
2207 2214
2208# Need a space before open parenthesis after if, while etc 2215# Need a space before open parenthesis after if, while etc
2209 if ($line=~/\b(if|while|for|switch)\(/) { 2216 if ($line=~/\b(if|while|for|switch)\(/) {