diff options
author | Kirill Primak <vyivel@eclair.cafe> | 2023-07-26 10:57:02 +0300 |
---|---|---|
committer | Simon Ser <contact@emersion.fr> | 2023-08-21 16:30:29 +0000 |
commit | f0cc712af14d9eae7cd8c56e74ae3b2547ff8ea2 (patch) | |
tree | db37162f225a4638d905fb1e38e6233d1f54fca6 | |
parent | ce615a44c06ed10ab51b9c5a630acb43ef014efb (diff) |
xdg-shell: don't return anything from create_xdg_surface()
-rw-r--r-- | include/types/wlr_xdg_shell.h | 3 | ||||
-rw-r--r-- | 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) { |