Hi Andy,
I just started using checkpatch and I was confused by the following WARN
message:
no space between function name and open parenthesis
I thought the probelm was that a space was missing while the truth is the
opposite.
How about the following patch?
--- checkpatch.pl.old 2008-01-04 13:37:51.000000000 +0100
+++ checkpatch.pl 2008-01-04 13:37:24.000000000 +0100
@@ -1117,7 +1117,7 @@
while ($line =~ /($Ident)\s+\(/g) {
if ($1 !~
/^(?:if|for|while|switch|return|volatile|__volatile__|__attribute__|format|__extension__|Copyright|case)$/
&&
$line !~ /$Type\s+\(/ && $line !~
/^.\#\s*define\b/) {
- WARN("no space between function name and open
parenthesis '('\n" . $herecurr);
+ WARN("don't put a space between function name
and open parenthesis '('\n" . $herecurr);
}
}
# Check operator spacing.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
