From f0cc712af14d9eae7cd8c56e74ae3b2547ff8ea2 Mon Sep 17 00:00:00 2001 From: Kirill Primak Date: Wed, 26 Jul 2023 10:57:02 +0300 Subject: xdg-shell: don't return anything from create_xdg_surface() --- include/types/wlr_xdg_shell.h | 3 +-- types/xdg_shell/wlr_xdg_surface.c | 11 ++++------- 2 files changed, 5 insertions(+), 9 deletions(-) diff --git a/include/types/wlr_xdg_shell.h b/include/types/wlr_xdg_shell.h index 0386388b..96cc117f 100644 --- a/include/types/wlr_xdg_shell.h +++ b/include/types/wlr_xdg_shell.h @@ -8,8 +8,7 @@ extern const struct wlr_surface_role xdg_toplevel_surface_role; extern const struct wlr_surface_role xdg_popup_surface_role; -struct wlr_xdg_surface *create_xdg_surface( - struct wlr_xdg_client *client, struct wlr_surface *wlr_surface, +void create_xdg_surface(struct wlr_xdg_client *client, struct wlr_surface *wlr_surface, uint32_t id); void destroy_xdg_surface(struct wlr_xdg_surface *surface); void destroy_xdg_surface_role_object(struct wlr_xdg_surface *surface); diff --git a/types/xdg_shell/wlr_xdg_surface.c b/types/xdg_shell/wlr_xdg_surface.c index d0ac4287..eec3a495 100644 --- a/types/xdg_shell/wlr_xdg_surface.c +++ b/types/xdg_shell/wlr_xdg_surface.c @@ -326,14 +326,13 @@ void xdg_surface_role_destroy(struct wlr_surface *wlr_surface) { free(surface); } -struct wlr_xdg_surface *create_xdg_surface( - struct wlr_xdg_client *client, struct wlr_surface *wlr_surface, +void create_xdg_surface(struct wlr_xdg_client *client, struct wlr_surface *wlr_surface, uint32_t id) { struct wlr_xdg_surface *surface = calloc(1, sizeof(struct wlr_xdg_surface)); if (surface == NULL) { wl_client_post_no_memory(client->client); - return NULL; + return; } surface->client = client; @@ -345,7 +344,7 @@ struct wlr_xdg_surface *create_xdg_surface( if (surface->resource == NULL) { free(surface); wl_client_post_no_memory(client->client); - return NULL; + return; } if (wlr_surface_has_buffer(surface->surface)) { @@ -354,7 +353,7 @@ struct wlr_xdg_surface *create_xdg_surface( wl_resource_post_error(client->resource, XDG_SURFACE_ERROR_UNCONFIGURED_BUFFER, "xdg_surface must not have a buffer at creation"); - return NULL; + return; } wl_list_init(&surface->configure_list); @@ -376,8 +375,6 @@ struct wlr_xdg_surface *create_xdg_surface( &xdg_surface_implementation, surface, xdg_surface_handle_resource_destroy); wl_list_insert(&client->surfaces, &surface->link); - - return surface; } void destroy_xdg_surface_role_object(struct wlr_xdg_surface *surface) { -- cgit v1.2.3