On 8/24/07, Andy Whitcroft <[EMAIL PROTECTED]> wrote: > 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.
I sent a reply accidentally only to Mike and not the list. I think the error message is wrong. That is really ugly code. Linux Kernel code believes in C not preprocessor tricks, so why would you need this? Who uses code like this, by the way? -sb - 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/