[PATCH 1/5] dix: don't InitXTestDevices if there's no XTest extension

[Julien Cristau]

On Tue, Nov  1, 2011 at 11:12:34 -0200, przanoni at gmail.com wrote:

> From: Paulo Zanoni <paulo.r.zanoni at intel.com>
> 
> Signed-off-by: Paulo Zanoni <paulo.r.zanoni at intel.com>
> ---
>  dix/devices.c |    5 ++++-
>  1 files changed, 4 insertions(+), 1 deletions(-)
> 
> diff --git a/dix/devices.c b/dix/devices.c
> index 673a360..9d67c06 100644
> --- a/dix/devices.c
> +++ b/dix/devices.c
> @@ -681,7 +681,10 @@ InitCoreDevices(void)
>          !EnableDevice(inputInfo.keyboard, TRUE))
>          FatalError("Failed to enable core devices.");
>  
> -    InitXTestDevices();
> +#ifdef XTEST
> +    if (!noTestExtensions)
> +	InitXTestDevices();
> +#endif
>  }
>  
>  /**

XTEST is always defined these days.  Also, IIRC there's a bunch of stuff
in XI that relies on the xtest devices.  Does that still work if you
don't init the devices?

Cheers,
Julien