compositor: do not recompute size on pointer_set_cursor
authorPekka Paalanen <pekka.paalanen@collabora.co.uk>
Tue, 26 Nov 2013 17:19:42 +0000 (18:19 +0100)
committerKristian Høgsberg <krh@bitplanet.net>
Thu, 28 Nov 2013 22:14:08 +0000 (14:14 -0800)
This removes the calls to weston_surface_buffer_width/height() from
input.c, which are the last external calls to them.

Instead, use the cached values from weston_surface::width,height. These
have already been set by weston_surface_commit(), because that is the
only way a weston_surface can get a buffer.

src/input.c

index ae0e832..f965ba1 100644 (file)
@@ -1495,8 +1495,8 @@ pointer_set_cursor(struct wl_client *client, struct wl_resource *resource,
        pointer->hotspot_y = y;
 
        if (surface->buffer_ref.buffer)
-               pointer_cursor_surface_configure(surface, 0, 0, weston_surface_buffer_width(surface),
-                                                               weston_surface_buffer_height(surface));
+               pointer_cursor_surface_configure(surface, 0, 0, surface->width,
+                                                               surface->height);
 }
 
 static void