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

Aaron Plattner aplattner at nvidia.com
Mon Sep 19 22:12:48 PDT 2011


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.

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

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
> 


More information about the xorg-devel mailing list