[PATCH weston v2] xwm: flush xcb connection only when we processed some event

[Bryce Harrington]

On Wed, Aug 12, 2015 at 09:55:12AM +0200, Marek Chalupa wrote:
> xwayland source is checked, so it dispatches twice on any event.
> If the other turn has no events to dispatch, we flush the connection
> redundantly
> 
> v2. do not flood logs with 'unhandled event' messages
> 
> Signed-off-by: Marek Chalupa <mchqwerty at gmail.com>
> Reviewed-by: Derek Foreman <derekf at osg.samsung.com>
Reviewed-by: Bryce Harrington <bryce at osg.samsung.com>

Thanks, pushed:
remote: I: patch #56878 updated using rev a1f3f3c9f209be52d19793d36bbda68ca24a714e
remote: I: 1 patch(es) updated to state Accepted.
To ssh://git.freedesktop.org/git/wayland/weston
   43af680..a1f3f3c  master -> master

> ---
>  xwayland/window-manager.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/xwayland/window-manager.c b/xwayland/window-manager.c
> index f9544d8..db6b437 100644
> --- a/xwayland/window-manager.c
> +++ b/xwayland/window-manager.c
> @@ -1979,7 +1979,8 @@ weston_wm_handle_event(int fd, uint32_t mask, void *data)
>  		count++;
>  	}
>  
> -	xcb_flush(wm->conn);
> +	if (count != 0)
> +		xcb_flush(wm->conn);
>  
>  	return count;
>  }
> -- 
> 2.4.3
> 
> _______________________________________________
> wayland-devel mailing list
> wayland-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/wayland-devel