wayland-drm: remove wl_buffer.damage
authorPekka Paalanen <ppaalanen@gmail.com>
Tue, 10 Apr 2012 12:35:06 +0000 (15:35 +0300)
committerKristian Høgsberg <krh@bitplanet.net>
Tue, 10 Apr 2012 17:06:56 +0000 (13:06 -0400)
This is a related fix for the Wayland change:

commit 83685c506e76212ae4e5cb722205d98d3b0603b9
Author: Kristian Høgsberg <krh@bitplanet.net>
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 <ppaalanen@gmail.com>
src/egl/wayland/wayland-drm/wayland-drm.c

index 42e6788d87a404f9d9883ce935d77c7289f0f2cd..101b2c4005131625456dcf7f2f4069cef6daaf70 100644 (file)
@@ -53,12 +53,6 @@ struct wl_drm_buffer {
        void *driver_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)
 {
@@ -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
 };