EXT: [PATCH weston 2/2] libweston: fix output reflow on removal

Ray, Ian (GE Healthcare) ian.ray at ge.com
Wed Jul 4 13:21:14 UTC 2018



> On 21 Jun 2018, at 15.53, Pekka Paalanen <ppaalanen at gmail.com> wrote:
> 
> From: Pekka Paalanen <pekka.paalanen at collabora.co.uk>
> 
> This is regression apparently introduced in
> 0de859ede4bad72b5d3b78e086632f02196d997f, which accidentally swapped the
> sign of 'delta_width' in the original call site. If one removes an
> output, the remaining outputs on the right are getting moved even
> further to the right.
> 
> The outputs to the right should be moved to the left instead, to close
> the gap left by the removed output.
> 

Reviewed-by: Ian Ray <ian.ray at ge.com>


> Reported-by: Tomasz Olszak <olszak.tomasz at gmail.com>
> Signed-off-by: Pekka Paalanen <pekka.paalanen at collabora.co.uk>
> ---
> libweston/compositor.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/libweston/compositor.c b/libweston/compositor.c
> index 8e01eacc..516be96b 100644
> --- a/libweston/compositor.c
> +++ b/libweston/compositor.c
> @@ -5409,7 +5409,7 @@ weston_compositor_remove_output(struct weston_output *output)
> 
> 	weston_presentation_feedback_discard_list(&output->feedback_list);
> 
> -	weston_compositor_reflow_outputs(compositor, output, output->width);
> +	weston_compositor_reflow_outputs(compositor, output, -output->width);
> 
> 	wl_list_remove(&output->link);
> 	wl_list_insert(compositor->pending_output_list.prev, &output->link);
> -- 
> 2.16.4
> 
> _______________________________________________
> wayland-devel mailing list
> wayland-devel at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/wayland-devel



More information about the wayland-devel mailing list