[PATCH libinput] evdev.c: Return from evdev_configure_device() after configuring a device as a tablet

[Peter Hutterer]

On Mon, Jan 19, 2015 at 08:54:45PM -0500, Stephen Chandler Paul wrote:
> As it turns out upon detecting a tablet in evdev_configure_device(), there isn't
> any call to return to escape the function once we've finished configuring the
> tablet. As a result, the code continues running and ends up reconfiguring the
> device as a pointer, resulting in strange behavior such as left-handed mode
> being enabled by default.
> 
> Signed-off-by: Stephen Chandler Paul <thatslyude at gmail.com>

thanks, applied and pushed.

Cheers,
   Peter

> ---
>  src/evdev.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/src/evdev.c b/src/evdev.c
> index 939d266..defd004 100644
> --- a/src/evdev.c
> +++ b/src/evdev.c
> @@ -1448,6 +1448,7 @@ evdev_configure_device(struct evdev_device *device)
>  			log_info(libinput,
>  				 "input device '%s', %s is a tablet\n",
>  				 device->devname, devnode);
> +			return device->dispatch == NULL ? -1 : 0;
>  		}
>  
>  		for (i = 0; i < KEY_MAX; i++) {
> -- 
> 2.0.5
>