On Tue,  7 Feb 2017 17:29:53 -0500
Micah Fedke <[email protected]> wrote:

> Signed-off-by: Micah Fedke <[email protected]>
> ---
>  src/egl/wayland/wayland-drm/wayland-drm.c | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/src/egl/wayland/wayland-drm/wayland-drm.c 
> b/src/egl/wayland/wayland-drm/wayland-drm.c
> index 4fc12521d2..5c2bb0369e 100644
> --- a/src/egl/wayland/wayland-drm/wayland-drm.c
> +++ b/src/egl/wayland/wayland-drm/wayland-drm.c
> @@ -55,7 +55,7 @@ struct wl_drm {
>  static void
>  destroy_buffer(struct wl_resource *resource)
>  {
> -     struct wl_drm_buffer *buffer = resource->data;
> +     struct wl_drm_buffer *buffer = wl_resource_get_user_data(resource);
>       struct wl_drm *drm = buffer->drm;
>  
>       drm->callbacks->release_buffer(drm->user_data, buffer);
> @@ -77,7 +77,7 @@ create_buffer(struct wl_client *client, struct wl_resource 
> *resource,
>                int32_t offset1, int32_t stride1,
>                int32_t offset2, int32_t stride2)
>  {
> -     struct wl_drm *drm = resource->data;
> +     struct wl_drm *drm = wl_resource_get_user_data(resource);
>       struct wl_drm_buffer *buffer;
>  
>       buffer = calloc(1, sizeof *buffer);
> @@ -187,7 +187,7 @@ static void
>  drm_authenticate(struct wl_client *client,
>                struct wl_resource *resource, uint32_t id)
>  {
> -     struct wl_drm *drm = resource->data;
> +     struct wl_drm *drm = wl_resource_get_user_data(resource);
>  
>       if (drm->callbacks->authenticate(drm->user_data, id) < 0)
>               wl_resource_post_error(resource,

Hi,

this looks good to me:
Reviewed-by: Pekka Paalanen <[email protected]>

It would also be good to add #define WL_HIDE_DEPRECATED to all
wayland-related .c files before #includes so that the compiler will
catch any attempts to use the deprecated definitions again.


Thanks,
pq

Attachment: pgpx8DG7zXeQ6.pgp
Description: OpenPGP digital signature

_______________________________________________
mesa-dev mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to