From dc8734ac443441f4c5ae3e8db234b812e6a707c9 Mon Sep 17 00:00:00 2001 From: "Enrico Weigelt, metux IT consult" Date: Wed, 21 Aug 2024 11:17:15 +0200 Subject: [PATCH] (submit/xnest-cleanup-event-handler) Xnest: split off event handler Tidy it up a bit and split the actual request handler from the request reading loop, making upcoming reworks easier to review (smaller diffs). Signed-off-by: Enrico Weigelt, metux IT consult --- hw/xnest/Events.c | 20 +++++++++++++------- 1 file changed, 13 insertions(+), 7 deletions(-) diff --git a/hw/xnest/Events.c b/hw/xnest/Events.c index 8ecbec095..02078107d 100644 --- a/hw/xnest/Events.c +++ b/hw/xnest/Events.c @@ -108,13 +108,10 @@ xnestQueueKeyEvent(int type, unsigned int keycode) QueueKeyboardEvents(xnestKeyboardDevice, type, keycode); } -void -xnestCollectEvents(void) +static void +xnest_handle_event(XEvent X) { - XEvent X; - - while (XCheckIfEvent(xnestDisplay, &X, xnestNotExposurePredicate, NULL)) { - switch (X.type) { + switch (X.type) { case KeyPress: { xnestUpdateModifierState(X.xkey.state); @@ -239,6 +236,15 @@ xnestCollectEvents(void) default: ErrorF("xnest warning: unhandled event: %d\n", X.type); break; - } + } +} + +void +xnestCollectEvents(void) +{ + XEvent X; + + while (XCheckIfEvent(xnestDisplay, &X, xnestNotExposurePredicate, NULL)) { + xnest_handle_event(X); } }