compositor: make wlr_compositor_destroy destroy resources

This commit is contained in:
emersion 2018-04-24 23:05:15 +01:00
parent cd9f25711c
commit 89a9c96fab
No known key found for this signature in database
GPG key ID: 0FDE7BE0E88F5E48
4 changed files with 39 additions and 24 deletions

View file

@ -15,7 +15,8 @@ struct wlr_compositor {
struct wl_global *wl_global; struct wl_global *wl_global;
struct wl_list wl_resources; struct wl_list wl_resources;
struct wlr_renderer *renderer; struct wlr_renderer *renderer;
struct wl_list surfaces; struct wl_list surface_resources;
struct wl_list region_resources;
struct wlr_subcompositor subcompositor; struct wlr_subcompositor subcompositor;

View file

@ -8,8 +8,7 @@ struct wl_resource;
/* /*
* Implements the given resource as region. * Implements the given resource as region.
*/ */
void wlr_region_create(struct wl_client *client, struct wl_resource *res, struct wl_resource *wlr_region_create(struct wl_client *client, uint32_t id);
uint32_t id);
pixman_region32_t *wlr_region_from_resource(struct wl_resource *resource); pixman_region32_t *wlr_region_from_resource(struct wl_resource *resource);

View file

@ -99,6 +99,10 @@ static void subcompositor_init(struct wlr_subcompositor *subcompositor,
static void subcompositor_finish(struct wlr_subcompositor *subcompositor) { static void subcompositor_finish(struct wlr_subcompositor *subcompositor) {
wl_global_destroy(subcompositor->wl_global); wl_global_destroy(subcompositor->wl_global);
struct wl_resource *resource, *tmp;
wl_resource_for_each_safe(resource, tmp, &subcompositor->wl_resources) {
wl_resource_destroy(resource);
}
} }
@ -111,7 +115,8 @@ static struct wlr_compositor *compositor_from_resource(
return wl_resource_get_user_data(resource); return wl_resource_get_user_data(resource);
} }
static void destroy_surface_listener(struct wl_listener *listener, void *data) { static void compositor_handle_surface_destroy(struct wl_listener *listener,
void *data) {
wl_list_remove(wl_resource_get_link(data)); wl_list_remove(wl_resource_get_link(data));
} }
@ -134,35 +139,36 @@ static void wl_compositor_create_surface(struct wl_client *client,
return; return;
} }
surface->compositor_data = compositor; surface->compositor_data = compositor;
surface->compositor_listener.notify = &destroy_surface_listener; surface->compositor_listener.notify = compositor_handle_surface_destroy;
wl_resource_add_destroy_listener(surface_resource, wl_resource_add_destroy_listener(surface_resource,
&surface->compositor_listener); &surface->compositor_listener);
wl_list_insert(&compositor->surfaces, wl_list_insert(&compositor->surface_resources,
wl_resource_get_link(surface_resource)); wl_resource_get_link(surface_resource));
wlr_signal_emit_safe(&compositor->events.new_surface, surface); wlr_signal_emit_safe(&compositor->events.new_surface, surface);
} }
static void wl_compositor_create_region(struct wl_client *client, static void wl_compositor_create_region(struct wl_client *client,
struct wl_resource *resource, uint32_t id) { struct wl_resource *resource, uint32_t id) {
wlr_region_create(client, resource, id); struct wlr_compositor *compositor = compositor_from_resource(resource);
struct wl_resource *region_resource = wlr_region_create(client, id);
if (region_resource == NULL) {
wl_resource_post_no_memory(resource);
return;
}
wl_list_insert(&compositor->region_resources,
wl_resource_get_link(region_resource));
} }
static const struct wl_compositor_interface wl_compositor_impl = { static const struct wl_compositor_interface wl_compositor_impl = {
.create_surface = wl_compositor_create_surface, .create_surface = wl_compositor_create_surface,
.create_region = wl_compositor_create_region .create_region = wl_compositor_create_region,
}; };
static void wl_compositor_destroy(struct wl_resource *resource) { static void wl_compositor_destroy(struct wl_resource *resource) {
struct wlr_compositor *compositor = compositor_from_resource(resource); wl_list_remove(wl_resource_get_link(resource));
struct wl_resource *_resource = NULL;
wl_resource_for_each(_resource, &compositor->wl_resources) {
if (_resource == resource) {
struct wl_list *link = wl_resource_get_link(_resource);
wl_list_remove(link);
break;
}
}
} }
static void wl_compositor_bind(struct wl_client *wl_client, void *data, static void wl_compositor_bind(struct wl_client *wl_client, void *data,
@ -189,6 +195,16 @@ void wlr_compositor_destroy(struct wlr_compositor *compositor) {
subcompositor_finish(&compositor->subcompositor); subcompositor_finish(&compositor->subcompositor);
wl_list_remove(&compositor->display_destroy.link); wl_list_remove(&compositor->display_destroy.link);
wl_global_destroy(compositor->wl_global); wl_global_destroy(compositor->wl_global);
struct wl_resource *resource, *tmp;
wl_resource_for_each_safe(resource, tmp, &compositor->surface_resources) {
wl_resource_destroy(resource);
}
wl_resource_for_each_safe(resource, tmp, &compositor->region_resources) {
wl_resource_destroy(resource);
}
wl_resource_for_each_safe(resource, tmp, &compositor->wl_resources) {
wl_resource_destroy(resource);
}
free(compositor); free(compositor);
} }
@ -218,7 +234,8 @@ struct wlr_compositor *wlr_compositor_create(struct wl_display *display,
compositor->renderer = renderer; compositor->renderer = renderer;
wl_list_init(&compositor->wl_resources); wl_list_init(&compositor->wl_resources);
wl_list_init(&compositor->surfaces); wl_list_init(&compositor->surface_resources);
wl_list_init(&compositor->region_resources);
wl_signal_init(&compositor->events.new_surface); wl_signal_init(&compositor->events.new_surface);
wl_signal_init(&compositor->events.destroy); wl_signal_init(&compositor->events.destroy);

View file

@ -38,12 +38,10 @@ static void destroy_region(struct wl_resource *resource) {
free(reg); free(reg);
} }
void wlr_region_create(struct wl_client *client, struct wl_resource *res, struct wl_resource *wlr_region_create(struct wl_client *client, uint32_t id) {
uint32_t id) {
pixman_region32_t *region = calloc(1, sizeof(pixman_region32_t)); pixman_region32_t *region = calloc(1, sizeof(pixman_region32_t));
if (region == NULL) { if (region == NULL) {
wl_resource_post_no_memory(res); return NULL;
return;
} }
pixman_region32_init(region); pixman_region32_init(region);
@ -52,11 +50,11 @@ void wlr_region_create(struct wl_client *client, struct wl_resource *res,
&wl_region_interface, 1, id); &wl_region_interface, 1, id);
if (region_resource == NULL) { if (region_resource == NULL) {
free(region); free(region);
wl_resource_post_no_memory(res); return NULL;
return;
} }
wl_resource_set_implementation(region_resource, &region_impl, region, wl_resource_set_implementation(region_resource, &region_impl, region,
destroy_region); destroy_region);
return region_resource;
} }
pixman_region32_t *wlr_region_from_resource(struct wl_resource *resource) { pixman_region32_t *wlr_region_from_resource(struct wl_resource *resource) {