<div dir="ltr">Nowhere in Weston. But i thought it was the accepted policy to not have around links not initted. But if it not it's no problem.<div><br></div><div style>Giulio</div></div><div class="gmail_extra"><br><br>
<div class="gmail_quote">2013/4/4 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 Thu, Apr 04, 2013 at 11:08:22PM +0200, Giulio Camuffo wrote:<br>
> ---<br>
>  src/compositor.c | 1 +<br>
>  1 file changed, 1 insertion(+)<br>
<br>
</div>When is this needed?  I don't think we rely on layer_link being valid<br>
when a surface is not mapped (not in any layer).<br>
<br>
Kristian<br>
<div class="im"><br>
> diff --git a/src/compositor.c b/src/compositor.c<br>
> index 3645b04..88f083c 100644<br>
> --- a/src/compositor.c<br>
> +++ b/src/compositor.c<br>
> @@ -977,6 +977,7 @@ weston_surface_unmap(struct weston_surface *surface)<br>
>       weston_surface_damage_below(surface);<br>
>       surface->output = NULL;<br>
>       wl_list_remove(&surface->layer_link);<br>
> +     wl_list_init(&surface->layer_link);<br>
><br>
>       wl_list_for_each(seat, &surface->compositor->seat_list, link) {<br>
>               if (seat->seat.keyboard &&<br>
> --<br>
> 1.8.2<br>
><br>
</div>> _______________________________________________<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>
</blockquote></div><br></div>