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

Marek Chalupa mchqwerty at gmail.com
Wed Aug 12 00:55:12 PDT 2015


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>
---
 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



More information about the wayland-devel mailing list