[patch] rename macro parameter list in cpufunc.h
Marius Strobl
marius at alchemy.franken.de
Tue Dec 28 17:35:31 UTC 2010
On Tue, Dec 28, 2010 at 02:27:04PM +0100, Pietro Cerutti wrote:
> Dear all,
>
> sys/sparc64/include/cpufunc.h defines macro having a parameter named
> 'xor'. This is a reserved keyword in C++. For this reason, it is
> impossible to use this header in C++ (e.g., by including
> machine/atomic.h).
>
> This patch [1] renames these parameters to _xor. Please review and/or
> approve (I need an explicit approval in order to commit to src).
>
> Thanks!
>
> Best Regards,
>
> [1] http://people.freebsd.org/~gahr/cpufunc.h.diff
Hrm, I really don't like the inconsistency only renaming "xor"
introduces, rototilling the whole file also seems excessive though
(also that would get it closer to style(9)). Would wrapping it in
__BEGIN_DECLS or something like that work as an alternate solution?
Marius
More information about the freebsd-sparc64
mailing list