Hi, It looks good.
Reviewed-by: Emre Ucan <eu...@de.adit-jv.com> Best regards Emre Ucan Software Group I (ADITG/SW1) Tel. +49 5121 49 6937 > -----Original Message----- > From: wayland-devel [mailto:wayland-devel- > boun...@lists.freedesktop.org] On Behalf Of Quentin Glidic > Sent: Dienstag, 5. Juli 2016 20:45 > To: wayland-devel@lists.freedesktop.org > Subject: [PATCH weston] libweston: Implement wl_output version 3 (release > request) > > From: Quentin Glidic <sardemff7+...@sardemff7.net> > > Signed-off-by: Quentin Glidic <sardemff7+...@sardemff7.net> > --- > libweston/compositor.c | 15 +++++++++++++-- > 1 file changed, 13 insertions(+), 2 deletions(-) > > diff --git a/libweston/compositor.c b/libweston/compositor.c > index 771f1c9..38f78fd 100644 > --- a/libweston/compositor.c > +++ b/libweston/compositor.c > @@ -4006,6 +4006,17 @@ weston_compositor_stack_plane(struct > weston_compositor *ec, > wl_list_insert(&ec->plane_list, &plane->link); > } > > +static void > +output_release(struct wl_client *client, struct wl_resource *resource) > +{ > + wl_resource_destroy(resource); > +} > + > +static const struct wl_output_interface output_interface = { > + output_release, > +}; > + > + > static void unbind_resource(struct wl_resource *resource) > { > wl_list_remove(wl_resource_get_link(resource)); > @@ -4027,7 +4038,7 @@ bind_output(struct wl_client *client, > } > > wl_list_insert(&output->resource_list, > wl_resource_get_link(resource)); > - wl_resource_set_implementation(resource, NULL, data, > unbind_resource); > + wl_resource_set_implementation(resource, &output_interface, > data, unbind_resource); > > wl_output_send_geometry(resource, > output->x, > @@ -4300,7 +4311,7 @@ weston_output_init(struct weston_output > *output, struct weston_compositor *c, > output->compositor->output_id_pool |= 1u << output->id; > > output->global = > - wl_global_create(c->wl_display, &wl_output_interface, 2, > + wl_global_create(c->wl_display, &wl_output_interface, 3, > output, bind_output); > } > > -- > 2.9.0 > > _______________________________________________ > wayland-devel mailing list > wayland-devel@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/wayland-devel _______________________________________________ wayland-devel mailing list wayland-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/wayland-devel