[PATCH weston 2/2] input: Unlink saved kbd focus listener when releasing seat

Bryce W. Harrington b.harrington at samsung.com
Fri Jan 10 12:51:14 PST 2014


For both patches:

Reviewed-by: Bryce Harrington <b.harrington at samsung.com>

On Fri, Jan 03, 2014 at 07:46:51PM +0100, Jonas Ådahl wrote:
> Not doing this would leave a invalid list item in the view's destroy
> signal listener list if destroying a seat that had previously lost
> keyboard focus.
> 
> Signed-off-by: Jonas Ådahl <jadahl at gmail.com>
> ---
>  src/input.c | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/src/input.c b/src/input.c
> index 07e9d6c..581326d 100644
> --- a/src/input.c
> +++ b/src/input.c
> @@ -2185,6 +2185,9 @@ weston_seat_release(struct weston_seat *seat)
>  {
>  	wl_list_remove(&seat->link);
>  
> +	if (seat->saved_kbd_focus)
> +		wl_list_remove(&seat->saved_kbd_focus_listener.link);
> +
>  	if (seat->pointer)
>  		weston_pointer_destroy(seat->pointer);
>  	if (seat->keyboard)
> -- 
> 1.8.3.2
> 
> _______________________________________________
> wayland-devel mailing list
> wayland-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/wayland-devel


More information about the wayland-devel mailing list