[PATCH weston] compositor: init the layer_link after removing it

[Giulio Camuffo]

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.

Giulio


2013/4/4 Kristian Høgsberg <hoegsberg at gmail.com>

> On Thu, Apr 04, 2013 at 11:08:22PM +0200, Giulio Camuffo wrote:
> > ---
> >  src/compositor.c | 1 +
> >  1 file changed, 1 insertion(+)
>
> When is this needed?  I don't think we rely on layer_link being valid
> when a surface is not mapped (not in any layer).
>
> Kristian
>
> > diff --git a/src/compositor.c b/src/compositor.c
> > index 3645b04..88f083c 100644
> > --- a/src/compositor.c
> > +++ b/src/compositor.c
> > @@ -977,6 +977,7 @@ weston_surface_unmap(struct weston_surface *surface)
> >       weston_surface_damage_below(surface);
> >       surface->output = NULL;
> >       wl_list_remove(&surface->layer_link);
> > +     wl_list_init(&surface->layer_link);
> >
> >       wl_list_for_each(seat, &surface->compositor->seat_list, link) {
> >               if (seat->seat.keyboard &&
> > --
> > 1.8.2
> >
> > _______________________________________________
> > wayland-devel mailing list
> > wayland-devel at lists.freedesktop.org
> > http://lists.freedesktop.org/mailman/listinfo/wayland-devel
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/wayland-devel/attachments/20130404/32f0593a/attachment-0001.html>