[PATCH weston v2] input: Use slightly less obscure logic in evdev_notify_keyboard_focus()

Pekka Paalanen ppaalanen at gmail.com
Thu Nov 27 05:30:39 PST 2014


On Wed, 19 Nov 2014 11:04:12 -0600
Derek Foreman <derekf at osg.samsung.com> wrote:

> While the test is actually correct (for non-negative numbers), it's not
> at all clear and seems to be an accidental order of operations mistake.
> 
> Also, add an assert() to make sure this number is never negative.
> 
> Closes bug 86346 - https://bugs.freedeskto.org/show_bug.cgi?id=86346
> 
> Signed-off-by: Derek Foreman <derekf at osg.samsung.com>
> ---
> 
> This version just moves the assert directly after the decrement...
> 
>  src/input.c           | 1 +
>  src/libinput-device.c | 2 +-
>  2 files changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/src/input.c b/src/input.c
> index 5f19875..cd5e10b 100644
> --- a/src/input.c
> +++ b/src/input.c
> @@ -2203,6 +2203,7 @@ WL_EXPORT void
>  weston_seat_release_keyboard(struct weston_seat *seat)
>  {
>  	seat->keyboard_device_count--;
> +	assert(seat->keyboard_device_count >= 0);
>  	if (seat->keyboard_device_count == 0) {
>  		weston_keyboard_set_focus(seat->keyboard, NULL);
>  		weston_keyboard_cancel_grab(seat->keyboard);
> diff --git a/src/libinput-device.c b/src/libinput-device.c
> index 0e3f46d..8a48905 100644
> --- a/src/libinput-device.c
> +++ b/src/libinput-device.c
> @@ -470,7 +470,7 @@ evdev_notify_keyboard_focus(struct weston_seat *seat,
>  {
>  	struct wl_array keys;
>  
> -	if (!seat->keyboard_device_count > 0)
> +	if (seat->keyboard_device_count == 0)
>  		return;
>  
>  	wl_array_init(&keys);

Pushed with Marek's R-b.

Thanks,
pq


More information about the wayland-devel mailing list