Messages in this thread Patch in this message | | | Date | Tue, 01 Jan 2008 12:12:22 +0200 | From | Benny Halevy <> | Subject | [PATCH] checkpatch.pl: recognize the #elif preprocessor directive |
| |
checkpatch.pl does not recognize #elif as a preprocessor directive causing it to print bogus errors for, e.g.: ERROR: need consistent spacing around '&' (ctx:WxV) when the operator is not recognized as unary in this context.
for example:
void foo(void) { int x, y, z; void *p[1] = { #if defined(X) &x #elif defined(Y) &y #else &z #endif }; }
Signed-off-by: Benny Halevy <bhalevy@panasas.com> --- scripts/checkpatch.pl | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl index 579f50f..9911c17 100755 --- a/scripts/checkpatch.pl +++ b/scripts/checkpatch.pl @@ -534,7 +534,7 @@ sub annotate_values { $preprocessor = 1; $paren_type[$paren] = 'N'; - } elsif ($cur =~ /^(#\s*(?:ifdef|ifndef|if|else|endif))/o) { + } elsif ($cur =~ /^(#\s*(?:ifdef|ifndef|if|else|elif|endif))/o) { print "PRE($1)\n" if ($debug); $preprocessor = 1; $type = 'N';
| |