[PATCH weston 3/4] ivi-shell: don't expilicitly assign outputs to views
Ucan, Emre (ADITG/ESB)
eucan at de.adit-jv.com
Wed Feb 7 11:54:16 UTC 2018
> -----Original Message-----
> From: Pekka Paalanen [mailto:ppaalanen at gmail.com]
> Sent: Mittwoch, 7. Februar 2018 11:22
> To: Ucan, Emre (ADITG/ESB)
> Cc: wayland-devel at lists.freedesktop.org
> Subject: Re: [PATCH weston 3/4] ivi-shell: don't expilicitly assign outputs to
> On Fri, 26 Jan 2018 15:04:58 +0100
> Emre Ucan <eucan at de.adit-jv.com> wrote:
> > it is assigned in weston_view_assign_outputs
> > Signed-off-by: Emre Ucan <eucan at de.adit-jv.com>
> > ---
> > ivi-shell/ivi-layout.c | 4 ----
> > 1 file changed, 4 deletions(-)
> > diff --git a/ivi-shell/ivi-layout.c b/ivi-shell/ivi-layout.c
> > index 3c52ce1..4799c25 100644
> > --- a/ivi-shell/ivi-layout.c
> > +++ b/ivi-shell/ivi-layout.c
> > @@ -821,10 +821,6 @@ commit_screen_list(struct ivi_layout *layout)
> > weston_layer_entry_insert(&layout-
> > &ivi_view->view-
> > -
> > - ivi_view->view->output = iviscrn->output;
> I understand the removal of this.
> > - ivi_view->ivisurf->surface->is_mapped =
> > - ivi_view->view->is_mapped = true;
> However, I would think these are still necessary. Why are they not
> necessary here?
> I do not see any other place that would set is_mapped=true for the view
> and the surface.
Yes, you are right. They are necessary. It is weird for me that we have to set surface and view mappedness in shell plugin.
A surface is mapped if it has a mapped view. A view is mapped if it is in the view list of the compositor. Shell plugin should not take care of this.
Anyway, I will send a new modified patch. Is it ok for you that I send an update only for this patch ?
> > }
> > }
> > }
More information about the wayland-devel