[PATCH 3/3] xfree86: Fix the sdk headers to be multilib-safe

Adam Jackson ajax at redhat.com
Fri Jan 7 09:59:16 PST 2011


On Fri, 2011-01-07 at 00:51 +0100, Mark Kettenis wrote:

> > --- a/include/colormapst.h
> > +++ b/include/colormapst.h
> > @@ -103,12 +103,12 @@ typedef struct _ColormapRec
> >  {
> >      VisualPtr	pVisual;
> >      short	class;		/* PseudoColor or DirectColor */
> > -#if defined(_XSERVER64)
> > +#if defined(_LP64)
> >      short	pad0;
> >      XID		pad1;
> >  #endif
> >      XID		mid;		/* client's name for colormap */
> > -#if defined(_XSERVER64) && (X_BYTE_ORDER == X_LITTLE_ENDIAN)
> > +#if defined(_LP64) && (X_BYTE_ORDER == X_LITTLE_ENDIAN)
> >      XID		pad2;
> >  #endif
> >      ScreenPtr	pScreen;	/* screen map is associated with */
> 
> This bit puzzles me.  Can you explain why it is necessary if
> xorg-server.h continues to define _XSERVER64 when appropriate?

Not strictly necessary I suppose, but xorg-server.h doesn't get pulled
in by any other header, so, might as well rely on the compiler's
predefines instead.

> > @@ -187,4 +184,8 @@
> >  /* X Access Control Extension */
> >  #undef XACE
> >  
> > +#ifdef _LP64
> > +#define _XSERVER64 1
> > +#endif
> > +
> >  #endif /* _XORG_SERVER_H_ */
> 
> I suppose there is some configure magic somewhere that replaces #undef
> _XSERVER64 on LP64 platforms.  Is that code now redundant?

Indeed there is, and indeed it is.  Will resend with that taken out too.

- ajax



More information about the xorg-devel mailing list