[PATCH 9/9] dix: Remove arch awareness from servermd.h

Adam Jackson ajax at redhat.com
Thu May 22 07:34:20 PDT 2014


On Wed, 2014-05-21 at 17:12 -0700, Alan Coopersmith wrote:
> On 05/21/14 12:31 PM, Adam Jackson wrote:
> > +#if __BYTE_ORDER__ == __ORDER_LITTLE_ENDIAN__
> 
> > +#elif __BYTE_ORDER == __ORDER_BIG_ENDIAN__
> 
> Did you intend to have the trailing __ on the first BYTE_ORDER but not
> the second?

Hah, I did not.

> It'd be nice if we could use autoconf settings from configure.ac, but
> I don't see where the ENDIAN variable set by AC_C_BIGENDIAN() gets
> used for anything at the moment.   Short of that, I think I'll have
> to see what's needed to handle the Solaris Studio compilers.

Yeah fair, didn't even notice we were already using that.  Will rework
accordingly.

- ajax



More information about the xorg-devel mailing list