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

Aaron Plattner aplattner at nvidia.com
Thu Apr 22 12:04:24 PDT 2010


On Thu, Apr 22, 2010 at 11:15:39AM -0700, Tiago Vignatti wrote:
> Bizarre. This seems to never be used before.
> 
> Signed-off-by: Tiago Vignatti <tiago.vignatti at nokia.com>
> ---
>  dix/main.c                 |    1 -
>  include/scrnintstr.h       |    1 -
>  test/xi2/protocol-common.c |    1 -
>  test/xtest.c               |    1 -
>  4 files changed, 0 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..4c7a879 100644
> --- a/include/scrnintstr.h
> +++ b/include/scrnintstr.h
> @@ -615,7 +615,6 @@ typedef struct _ScreenInfo {
>      int		numPixmapFormats;
>      PixmapFormatRec
>  		formats[MAXFORMATS];
> -    int		arraySize;

This breaks the ABI again.  Could you please either bump the ABI numbers in
hw/xfree86/common/xf86Module.h and remove the 'int unused' field below or
(preferably) just rename the field to "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


More information about the xorg-devel mailing list