[PATCH 1/2] xnest: Replace use of Colormap->devPriv with standard devPrivates.

Tiago Vignatti tiago.vignatti at nokia.com
Wed Jul 14 06:30:46 PDT 2010


On Wed, Jul 14, 2010 at 12:37:07AM +0200, ext Jamey Sharp wrote:
> Signed-off-by: Jamey Sharp <jamey at minilop.net>
> ---
>  hw/xnest/Color.c  |    5 ++---
>  hw/xnest/Color.h  |    3 ++-
>  hw/xnest/Screen.c |    2 ++
>  3 files changed, 6 insertions(+), 4 deletions(-)
> 
> diff --git a/hw/xnest/Color.c b/hw/xnest/Color.c
> index 883b77c..0aae8e9 100644
> --- a/hw/xnest/Color.c
> +++ b/hw/xnest/Color.c
> @@ -34,6 +34,8 @@ is" without express or implied warranty.
>  #include "XNWindow.h"
>  #include "Args.h"
>  
> +DevPrivateKeyRec xnestColormapPrivateKeyRec;
> +
>  static DevPrivateKeyRec cmapScrPrivateKeyRec;
>  #define cmapScrPrivateKey (&cmapScrPrivateKeyRec)
>  
> @@ -52,8 +54,6 @@ xnestCreateColormap(ColormapPtr pCmap)
>    pVisual = pCmap->pVisual;
>    ncolors = pVisual->ColormapEntries;
>  
> -  pCmap->devPriv = (pointer)malloc(sizeof(xnestPrivColormap));
> -  
>    xnestColormapPriv(pCmap)->colormap = 
>      XCreateColormap(xnestDisplay,
>  		    xnestDefaultWindows[pCmap->pScreen->myNum],
> @@ -130,7 +130,6 @@ void
>  xnestDestroyColormap(ColormapPtr pCmap)
>  {
>    XFreeColormap(xnestDisplay, xnestColormap(pCmap));
> -  free(pCmap->devPriv);
>  }
>  
>  #define SEARCH_PREDICATE \
> diff --git a/hw/xnest/Color.h b/hw/xnest/Color.h
> index f00bde4..d39c638 100644
> --- a/hw/xnest/Color.h
> +++ b/hw/xnest/Color.h
> @@ -32,8 +32,9 @@ typedef struct {
>    int index;
>  } xnestInstalledColormapWindows;
>  
> +extern DevPrivateKeyRec xnestColormapPrivateKeyRec;
>  #define xnestColormapPriv(pCmap) \
> -  ((xnestPrivColormap *)((pCmap)->devPriv))
> +  ((xnestPrivColormap *) dixLookupPrivate(&(pCmap)->devPrivates, &xnestColormapPrivateKeyRec))
>  
>  #define xnestColormap(pCmap) (xnestColormapPriv(pCmap)->colormap)
>  
> diff --git a/hw/xnest/Screen.c b/hw/xnest/Screen.c
> index 957fe4c..0a6cab6 100644
> --- a/hw/xnest/Screen.c
> +++ b/hw/xnest/Screen.c
> @@ -151,6 +151,8 @@ xnestOpenScreen(int index, ScreenPtr pScreen, int argc, char *argv[])
>      return FALSE;
>    if (!dixRegisterPrivateKey(&xnestPixmapPrivateKeyRec, PRIVATE_PIXMAP, sizeof (xnestPrivPixmap)))
>        return FALSE;
> +  if (!dixRegisterPrivateKey(&xnestColormapPrivateKeyRec, PRIVATE_COLORMAP, sizeof (xnestPrivColormap)))
> +      return FALSE;
>    if (!dixRegisterPrivateKey(&xnestCursorScreenKeyRec, PRIVATE_SCREEN, 0))
>        return FALSE;
>  

Reviewed-by: Tiago Vignatti <tiago.vignatti at nokia.com>

             Tiago


More information about the xorg-devel mailing list