On Fri, 26 Jan 2018 15:04:58 +0100
Emre Ucan <eu...@de.adit-jv.com> wrote:

> it is assigned in weston_view_assign_outputs
> 
> Signed-off-by: Emre Ucan <eu...@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->layout_layer.view_list,
>                                                         
> &ivi_view->view->layer_link);
> -
> -                             ivi_view->view->output = iviscrn->output;

Hi,

I understand the removal of this.

> -                             ivi_view->ivisurf->surface->is_mapped = true;
> -                             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.

>                       }
>               }
>       }

Thanks,
pq

Attachment: pgp8F8Ejq2ttX.pgp
Description: OpenPGP digital signature

_______________________________________________
wayland-devel mailing list
wayland-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/wayland-devel

Reply via email to