Re: kernel coding style for if ... else which cross #ifdef

From: Sam Ravnborg
Date: Sat May 24 2008 - 01:43:21 EST


> *However*, the best would really be if we changed Kconfig to emit
> configuration constants what were 0/1 instead of undefined/defined.
> That way we could do:
>
> if (CONFIG_SOMETHING && foo) {
> /* ... something ... */
> } else if ((mode & S_IWUGO) == 0) {
> /* ... */

We could do that - but then it would need another
name not to clash with all the places where we rely
on CONFIG_FOO='n' => CONFIG_FOO is not defined.

We could teach kconfig to emit something like:
#define KFOO 0 (for the 'n' value)
And 1 or 2 for the y and m values.

Sam
--
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/