Re: false positive in checkpatch.pl (complex macro values)

From: Andy Whitcroft
Date: Fri Aug 24 2007 - 07:41:48 EST


Mike Frysinger wrote:
> in some code that does like:
> #define foo { a, b, c, \
> d, e, f, g }
> ...
> int boo[] = foo;
> ...
>
> checkpatch.pl throws a fit:
> ERROR: Macros with complex values should be enclosed in parenthesis
> #10: FILE: ...
> +#define foo {a, b, c, d}
>
> perhaps the check should also allow {...} ? or ignore lists like this ...
> -mike

Ok, we can add that to the check. Next update will allow that.

Thanks for the report.

-apw
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/