For the whole series: Reviewed-by: Giulio Camuffo <[email protected]>
2015-11-05 21:57 GMT+02:00 Derek Foreman <[email protected]>: > From irc: > <pq> it creates a wl_buffer object in a way that no client can ever > access the storage. > > So, let's replace it with return NULL; and mark it with attribute > deprecated in the header. > > Reviewed-by: Pekka Paalanen <[email protected]> > Signed-off-by: Derek Foreman <[email protected]> > --- > > Change from v1: return NULL instead of abort()ing > > src/wayland-server-core.h | 2 +- > src/wayland-shm.c | 29 +---------------------------- > 2 files changed, 2 insertions(+), 29 deletions(-) > > diff --git a/src/wayland-server-core.h b/src/wayland-server-core.h > index 73f2dd2..85b4b9e 100644 > --- a/src/wayland-server-core.h > +++ b/src/wayland-server-core.h > @@ -472,7 +472,7 @@ wl_display_add_shm_format(struct wl_display *display, > uint32_t format); > struct wl_shm_buffer * > wl_shm_buffer_create(struct wl_client *client, > uint32_t id, int32_t width, int32_t height, > - int32_t stride, uint32_t format); > + int32_t stride, uint32_t format) WL_DEPRECATED; > > void wl_log_set_handler_server(wl_log_func_t handler); > > diff --git a/src/wayland-shm.c b/src/wayland-shm.c > index 1ab8f89..a05a8a0 100644 > --- a/src/wayland-shm.c > +++ b/src/wayland-shm.c > @@ -319,34 +319,7 @@ wl_shm_buffer_create(struct wl_client *client, > uint32_t id, int32_t width, int32_t height, > int32_t stride, uint32_t format) > { > - struct wl_shm_buffer *buffer; > - > - if (!format_is_supported(client, format)) > - return NULL; > - > - buffer = malloc(sizeof *buffer + stride * height); > - if (buffer == NULL) > - return NULL; > - > - buffer->width = width; > - buffer->height = height; > - buffer->format = format; > - buffer->stride = stride; > - buffer->offset = 0; > - buffer->pool = NULL; > - > - buffer->resource = > - wl_resource_create(client, &wl_buffer_interface, 1, id); > - if (buffer->resource == NULL) { > - free(buffer); > - return NULL; > - } > - > - wl_resource_set_implementation(buffer->resource, > - &shm_buffer_interface, > - buffer, destroy_buffer); > - > - return buffer; > + return NULL; > } > > WL_EXPORT struct wl_shm_buffer * > -- > 2.6.1 > _______________________________________________ wayland-devel mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/wayland-devel
