frederik
/
xserver
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Revert "Bug #6924: Restore the ABI for DrawableRec and ColormapRec to the state" 

This reverts commit bc0c56c407 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@minilop.net>
Reviewed-by: Aaron Plattner <aplattner@nvidia.com>
This commit is contained in:
Jamey Sharp 2010-07-10 09:28:47 -07:00
parent 87cc83474d
commit c96a43888c
2 changed files with 2 additions and 22 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
dix/colormap.c
View File

@ -287,13 +287,6 @@ CreateColormap (Colormap mid, ScreenPtr pScreen, VisualPtr pVisual,
if (!pmap) if (!pmap)
return BadAlloc; 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)); pmap->red = (EntryPtr)((char *)pmap + sizeof(ColormapRec));
sizebytes = size * sizeof(Entry); sizebytes = size * sizeof(Entry);
pmap->clientPixelsRed = (Pixel **)((char *)pmap->red + sizebytes); pmap->clientPixelsRed = (Pixel **)((char *)pmap->red + sizebytes);

17
include/colormapst.h
View File

@ -48,8 +48,6 @@ SOFTWARE.
#ifndef CMAPSTRUCT_H #ifndef CMAPSTRUCT_H
#define CMAPSTRUCT_H 1 #define CMAPSTRUCT_H 1
#include <X11/Xarch.h>
#include "colormap.h" #include "colormap.h"
#include "screenint.h" #include "screenint.h"
#include "privates.h" #include "privates.h"
@ -91,26 +89,15 @@ typedef struct _CMEntry
Bool fShared; Bool fShared;
} Entry; } 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 * only needs one cell table, we arbitrarily pick red. We keep track
* of that table with freeRed, numPixelsRed, and clientPixelsRed * of that table with freeRed, numPixelsRed, and clientPixelsRed */
*
* The padN variables are unfortunate ABI BC. See fdo bug #6924.
*/
typedef struct _ColormapRec typedef struct _ColormapRec
{ {
VisualPtr pVisual; VisualPtr pVisual;
short class; /* PseudoColor or DirectColor */ short class; /* PseudoColor or DirectColor */
#if defined(_LP64)
short pad0;
XID pad1;
#endif
XID mid; /* client's name for colormap */ 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 */ ScreenPtr pScreen; /* screen map is associated with */
short flags; /* 1 = IsDefault short flags; /* 1 = IsDefault
* 2 = AllAllocated */ * 2 = AllAllocated */