[PATCH 05/13] dix: Re-type and repack the head of ScreenRec

[Keith Packard]

On Tue, 23 Nov 2010 14:45:40 -0500, Adam Jackson <ajax at redhat.com> wrote:

>      int			myNum;	/* index of this instance in Screens[] */
> -    ATOM		id;
>      short		x, y, width, height;
>      short		mmWidth, mmHeight;
> -    short		numDepths;
> +    unsigned char	numDepths;
>      unsigned char      	rootDepth;
> -    DepthPtr       	allowedDepths;
> -    unsigned long      	rootVisual;
> -    unsigned long	defColormap;
>      short		minInstalledCmaps, maxInstalledCmaps;
>      char                backingStoreSupport, saveUnderSupport;
> -    unsigned long	whitePixel, blackPixel;
> +    DepthPtr       	allowedDepths;
> +    unsigned int      	rootVisual;
> +    unsigned int	defColormap;
> +    unsigned int	whitePixel, blackPixel;

Seems like these all have well defined existing types. Why aren't they
being used?

-- 
keith.packard at intel.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.x.org/archives/xorg-devel/attachments/20101123/b3dbe769/attachment.pgp>