On Wed, Sep 11, 2013 at 08:54:13PM +0200, Giulio Camuffo wrote: > with the surface ref-count feature a surface may live on after its > resource was destroyed. set it to NULL in that case, so that code > like find_resource_for_surface() in input.c will act accordingly.
That makes sense, applied. When you send out a v2 patch can you add a brief comment about what changed below the ---? > --- (that is, here) Kristian > src/compositor.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/src/compositor.c b/src/compositor.c > index 88df279..a79f911 100644 > --- a/src/compositor.c > +++ b/src/compositor.c > @@ -1071,6 +1071,9 @@ destroy_surface(struct wl_resource *resource) > struct weston_surface *surface = wl_resource_get_user_data(resource); > > weston_surface_destroy(surface); > + /* the surface may live so make sure nothing thinks > + it still has a resourc*e */ > + surface->resource = NULL; > } > > static void > -- > 1.8.4 > > _______________________________________________ > wayland-devel mailing list > [email protected] > http://lists.freedesktop.org/mailman/listinfo/wayland-devel _______________________________________________ wayland-devel mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/wayland-devel
