From: Pekka Paalanen Date: Tue, 10 Apr 2012 12:35:06 +0000 (+0300) Subject: wayland-drm: remove wl_buffer.damage X-Git-Tag: 062012170305~908 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=272bc489762c6946e3c0314e9e035b281c567310;p=profile%2Fivi%2Fmesa.git wayland-drm: remove wl_buffer.damage This is a related fix for the Wayland change: commit 83685c506e76212ae4e5cb722205d98d3b0603b9 Author: Kristian Høgsberg Date: Mon Mar 26 16:33:24 2012 -0400 Remove wl_buffer.damage and simplify shm implementation Apparently, this should also fix a memory leak. When wl_buffer.damage was removed from Wayland and Mesa was not fixed, wl_buffer.destroy ended up in the (empty) damage function instead of calling wl_resource_destroy(). Spotted during build as: CC wayland-drm-protocol.lo wayland-drm.c:80:2: warning: initialization from incompatible pointer type wayland-drm.c:82:1: warning: excess elements in struct initializer wayland-drm.c:82:1: warning: (near initialization for 'drm_buffer_interface') Signed-off-by: Pekka Paalanen --- diff --git a/src/egl/wayland/wayland-drm/wayland-drm.c b/src/egl/wayland/wayland-drm/wayland-drm.c index 42e6788..101b2c4 100644 --- a/src/egl/wayland/wayland-drm/wayland-drm.c +++ b/src/egl/wayland/wayland-drm/wayland-drm.c @@ -54,12 +54,6 @@ struct wl_drm_buffer { }; static void -buffer_damage(struct wl_client *client, struct wl_resource *buffer, - int32_t x, int32_t y, int32_t width, int32_t height) -{ -} - -static void destroy_buffer(struct wl_resource *resource) { struct wl_drm_buffer *buffer = resource->data; @@ -77,7 +71,6 @@ buffer_destroy(struct wl_client *client, struct wl_resource *resource) } const static struct wl_buffer_interface drm_buffer_interface = { - buffer_damage, buffer_destroy };