[PATCH 5/9] Revert "Bug #6924: Restore the ABI for DrawableRec and ColormapRec to the state"

Jamey Sharp jamey at minilop.net
Mon Sep 19 22:35:30 PDT 2011


On Mon, Sep 19, 2011 at 10:12:48PM -0700, Aaron Plattner wrote:
> Sure, why not?  This is the first ABI break since server-1.11 that I've
> seen go out to the list, I think, but it seems naive to assume there won't
> be more.

Oh, right--I wrote the commit message when I thought it would be merged
for 1.10, or possibly 1.9. I forget. But yeah, I'm pretty sure this
won't be the only ABI break of 1.12. I noticed that the IOADDRESS
work-in-progress breaks the ABI too, for example.

> Reviewed-by: Aaron Plattner <aplattner at nvidia.com>

Thanks!
Jamey

> On Sat, Sep 17, 2011 at 01:22:31AM -0700, Jamey Sharp wrote:
> > This reverts commit bc0c56c407117d1545e20d21f7d30eb3472d618b since we're
> > breaking ABI anyway.
> > 
> > Conflicts:
> > 
> > 	ChangeLog
> > 	dix/colormap.c
> > 	dix/pixmap.c
> > 	dix/window.c
> > 	include/pixmapstr.h
> > 
> > Signed-off-by: Jamey Sharp <jamey at minilop.net>
> > ---
> >  dix/colormap.c       |    7 -------
> >  include/colormapst.h |   17 ++---------------
> >  2 files changed, 2 insertions(+), 22 deletions(-)
> > 
> > diff --git a/dix/colormap.c b/dix/colormap.c
> > index 0e1feb6..038457d 100644
> > --- a/dix/colormap.c
> > +++ b/dix/colormap.c
> > @@ -287,13 +287,6 @@ CreateColormap (Colormap mid, ScreenPtr pScreen, VisualPtr pVisual,
> >  	if (!pmap)
> >  	    return BadAlloc;
> >      }
> > -#if defined(_XSERVER64)
> > -    pmap->pad0 = 0;
> > -    pmap->pad1 = 0;
> > -#if (X_BYTE_ORDER == X_LITTLE_ENDIAN)
> > -    pmap->pad2 = 0;
> > -#endif
> > -#endif
> >      pmap->red = (EntryPtr)((char *)pmap + sizeof(ColormapRec));    
> >      sizebytes = size * sizeof(Entry);
> >      pmap->clientPixelsRed = (Pixel **)((char *)pmap->red + sizebytes);
> > diff --git a/include/colormapst.h b/include/colormapst.h
> > index b597e2c..bb79c86 100644
> > --- a/include/colormapst.h
> > +++ b/include/colormapst.h
> > @@ -48,8 +48,6 @@ SOFTWARE.
> >  #ifndef CMAPSTRUCT_H
> >  #define CMAPSTRUCT_H 1
> >  
> > -#include <X11/Xarch.h>
> > -
> >  #include "colormap.h"
> >  #include "screenint.h"
> >  #include "privates.h"
> > @@ -91,26 +89,15 @@ typedef struct _CMEntry
> >      Bool	fShared;
> >  } Entry;
> >  
> > -/*
> > - * COLORMAPs can be used for either Direct or Pseudo color.  PseudoColor
> > +/* COLORMAPs can be used for either Direct or Pseudo color.  PseudoColor
> >   * only needs one cell table, we arbitrarily pick red.  We keep track
> > - * of that table with freeRed, numPixelsRed, and clientPixelsRed
> > - *
> > - * The padN variables are unfortunate ABI BC. See fdo bug #6924.
> > - */
> > + * of that table with freeRed, numPixelsRed, and clientPixelsRed */
> >  
> >  typedef struct _ColormapRec
> >  {
> >      VisualPtr	pVisual;
> >      short	class;		/* PseudoColor or DirectColor */
> > -#if defined(_LP64)
> > -    short	pad0;
> > -    XID		pad1;
> > -#endif
> >      XID		mid;		/* client's name for colormap */
> > -#if defined(_LP64) && (X_BYTE_ORDER == X_LITTLE_ENDIAN)
> > -    XID		pad2;
> > -#endif
> >      ScreenPtr	pScreen;	/* screen map is associated with */
> >      short	flags;		/* 1 = IsDefault
> >  				 * 2 = AllAllocated */
> > -- 
> > 1.7.5.4
> > 
> > _______________________________________________
> > xorg-devel at lists.x.org: X.Org development
> > Archives: http://lists.x.org/archives/xorg-devel
> > Info: http://lists.x.org/mailman/listinfo/xorg-devel
> > 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.x.org/archives/xorg-devel/attachments/20110919/7a63f56f/attachment.pgp>


More information about the xorg-devel mailing list