On Wed, 9 Oct 2002, Tim Hockin wrote:
> >
> > In other words, that __UID16 thing should be a real CONFIG_XXX option.
>
> Because Sparc64/s390x/? still need to tell highuid.h to do macro magic for
> NEW_TO_OLD_UID() and friends in some places and not others. A CONFIG_XXX
> applies all the time to all files.
If __UID16 works, then renaming it to CONFIG_UID16_ONLY _must_ also work.
I don't understand your argument about other architectures. I'm claiming
that __UID16 is a config option, and that it must be renamed to _show_
that it is a config option.
If the renaming results in code that doesn't work, then the code didn't
work in the first place or your cpp is incredibly broken.
WE MUST NOT HAVE CONFIG OPTIONS THAT ARE HIDDEN AND CALLED __UID16! That's
my whole point.
Linus
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
This archive was generated by hypermail 2b29 : Tue Oct 15 2002 - 22:00:33 EST