<div dir="ltr">Sure, thanks.</div><div class="gmail_extra"><br><br><div class="gmail_quote">2013/9/11 Kristian Høgsberg <span dir="ltr"><<a href="mailto:hoegsberg@gmail.com" target="_blank">hoegsberg@gmail.com</a>></span><br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On Wed, Sep 11, 2013 at 08:54:13PM +0200, Giulio Camuffo wrote:<br>
> with the surface ref-count feature a surface may live on after its<br>
> resource was destroyed. set it to NULL in that case, so that code<br>
> like find_resource_for_surface() in input.c will act accordingly.<br>
<br>
</div>That makes sense, applied.  When you send out a v2 patch can you add a<br>
brief comment about what changed below the ---?<br>
<br>
> ---<br>
<br>
(that is, here)<br>
<span class="HOEnZb"><font color="#888888"><br>
Kristian<br>
</font></span><div class="im HOEnZb"><br>
>  src/compositor.c | 3 +++<br>
>  1 file changed, 3 insertions(+)<br>
><br>
> diff --git a/src/compositor.c b/src/compositor.c<br>
> index 88df279..a79f911 100644<br>
> --- a/src/compositor.c<br>
> +++ b/src/compositor.c<br>
> @@ -1071,6 +1071,9 @@ destroy_surface(struct wl_resource *resource)<br>
>       struct weston_surface *surface = wl_resource_get_user_data(resource);<br>
><br>
>       weston_surface_destroy(surface);<br>
> +     /* the surface may live so make sure nothing thinks<br>
> +      it still has a resourc*e */<br>
> +     surface->resource = NULL;<br>
>  }<br>
><br>
>  static void<br>
> --<br>
> 1.8.4<br>
><br>
</div><div class="HOEnZb"><div class="h5">> _______________________________________________<br>
> wayland-devel mailing list<br>
> <a href="mailto:wayland-devel@lists.freedesktop.org">wayland-devel@lists.freedesktop.org</a><br>
> <a href="http://lists.freedesktop.org/mailman/listinfo/wayland-devel" target="_blank">http://lists.freedesktop.org/mailman/listinfo/wayland-devel</a><br>
</div></div></blockquote></div><br></div>