On Wed, Jun 03, 2015 at 03:53:28PM -0500, Derek Foreman wrote:
> We already have a pointer to the keyboard, so we can change all
> seat->keyboard to keyboard
>
> Signed-off-by: Derek Foreman <der...@osg.samsung.com>
> ---
> src/input.c | 22 +++++++++++-----------
> 1 file changed, 11 insertions(+), 11 deletions(-)
>
> diff --git a/src/input.c b/src/input.c
> index f8f19f7..dee3ca9 100644
> --- a/src/input.c
> +++ b/src/input.c
> @@ -1777,7 +1777,7 @@ seat_get_keyboard(struct wl_client *client, struct
> wl_resource *resource,
> struct weston_keyboard *keyboard = seat->keyboard;
> struct wl_resource *cr;
>
> - if (!seat->keyboard)
> + if (!keyboard)
> return;
>
> cr = wl_resource_create(client, &wl_keyboard_interface,
> @@ -1790,7 +1790,7 @@ seat_get_keyboard(struct wl_client *client, struct
> wl_resource *resource,
> /* May be moved to focused list later by either
> * weston_keyboard_set_focus or directly if this client is already
> * focused */
> - wl_list_insert(&seat->keyboard->resource_list,
> wl_resource_get_link(cr));
> + wl_list_insert(&keyboard->resource_list, wl_resource_get_link(cr));
> wl_resource_set_implementation(cr, &keyboard_interface,
> seat, unbind_resource);
>
> @@ -1813,27 +1813,27 @@ seat_get_keyboard(struct wl_client *client, struct
> wl_resource *resource,
> }
>
> if (should_send_modifiers_to_client(seat, client)) {
> - send_modifiers_to_resource(seat->keyboard,
> + send_modifiers_to_resource(keyboard,
> cr,
> - seat->keyboard->focus_serial);
> + keyboard->focus_serial);
> }
>
> - if (seat->keyboard->focus && seat->keyboard->focus->resource &&
> - wl_resource_get_client(seat->keyboard->focus->resource) == client) {
> + if (keyboard->focus && keyboard->focus->resource &&
> + wl_resource_get_client(keyboard->focus->resource) == client) {
> struct weston_surface *surface =
> - (struct weston_surface *) seat->keyboard->focus;
> + (struct weston_surface *)keyboard->focus;
What is our coding style regarding this? The removed way here is slightly
more common within weston, and overwhelmingly more common in wayland.
Either way, the rest of the patch is Reviewed-by: Jonas Ådahl
<jad...@gmail.com>
Jonas
>
> wl_list_remove(wl_resource_get_link(cr));
> - wl_list_insert(&seat->keyboard->focus_resource_list,
> + wl_list_insert(&keyboard->focus_resource_list,
> wl_resource_get_link(cr));
> wl_keyboard_send_enter(cr,
> - seat->keyboard->focus_serial,
> + keyboard->focus_serial,
> surface->resource,
> - &seat->keyboard->keys);
> + &keyboard->keys);
>
> /* If this is the first keyboard resource for this
> * client... */
> - if (seat->keyboard->focus_resource_list.prev ==
> + if (keyboard->focus_resource_list.prev ==
> wl_resource_get_link(cr))
> wl_data_device_set_keyboard_focus(seat);
> }
> --
> 2.1.4
>
> _______________________________________________
> wayland-devel mailing list
> wayland-devel@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/wayland-devel
_______________________________________________
wayland-devel mailing list
wayland-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/wayland-devel
