[PATCH v2] dix and others: remove unused arraySize field from ScreenInfo

Aaron Plattner aplattner at nvidia.com
Fri Apr 23 16:39:34 PDT 2010


On Fri, Apr 23, 2010 at 05:25:26AM -0700, Tiago Vignatti wrote:
> Bizarre. This seems to never be used before. I left the field in ScreenInfo,
> with another name. So, stop looking at it.
> 
> Signed-off-by: Tiago Vignatti <tiago.vignatti at nokia.com>
> ---
> - added aaron's suggestion.
> 
>  dix/main.c                 |    1 -
>  include/scrnintstr.h       |    2 +-
>  test/xi2/protocol-common.c |    1 -
>  test/xtest.c               |    1 -
>  4 files changed, 1 insertions(+), 4 deletions(-)
> 
> diff --git a/dix/main.c b/dix/main.c
> index f023536..e5667a6 100644
> --- a/dix/main.c
> +++ b/dix/main.c
> @@ -190,7 +190,6 @@ int main(int argc, char *argv[], char *envp[])
>  	    FatalError("couldn't init server resources");
>  
>  	SetInputCheck(&alwaysCheckForInput[0], &alwaysCheckForInput[1]);
> -	screenInfo.arraySize = MAXSCREENS;
>  	screenInfo.numScreens = 0;
>  
>  	InitAtoms();
> diff --git a/include/scrnintstr.h b/include/scrnintstr.h
> index c42119d..5a7c57d 100644
> --- a/include/scrnintstr.h
> +++ b/include/scrnintstr.h
> @@ -615,7 +615,7 @@ typedef struct _ScreenInfo {
>      int		numPixmapFormats;
>      PixmapFormatRec
>  		formats[MAXFORMATS];
> -    int		arraySize;
> +    int		unused2;
>      int		numScreens;
>      ScreenPtr	screens[MAXSCREENS];
>      int		unused;
> diff --git a/test/xi2/protocol-common.c b/test/xi2/protocol-common.c
> index 2dff1ad..0afa55a 100644
> --- a/test/xi2/protocol-common.c
> +++ b/test/xi2/protocol-common.c
> @@ -141,7 +141,6 @@ static Bool device_cursor_init(DeviceIntPtr dev, ScreenPtr screen) { return TRUE
>  static Bool set_cursor_pos(DeviceIntPtr dev, ScreenPtr screen, int x, int y, Bool event) { return TRUE; }
>  void init_simple(void)
>  {
> -    screenInfo.arraySize = MAXSCREENS;
>      screenInfo.numScreens = 1;
>      screenInfo.screens[0] = &screen;
>  
> diff --git a/test/xtest.c b/test/xtest.c
> index 572f5d2..cc1188d 100644
> --- a/test/xtest.c
> +++ b/test/xtest.c
> @@ -49,7 +49,6 @@ static void xtest_init_devices(void)
>  
>      /* random stuff that needs initialization */
>      memset(&screen, 0, sizeof(screen));
> -    screenInfo.arraySize = MAXSCREENS;
>      screenInfo.numScreens = 1;
>      screenInfo.screens[0] = &screen;
>      screen.myNum = 0;
> -- 
> 1.6.0.4

That looks better to me, thanks.

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


More information about the xorg-devel mailing list