[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