From d497375eaa8621e8edca9c51a7c209f61d433729 Mon Sep 17 00:00:00 2001
From: "Enrico Weigelt, metux IT consult" <info@metux.net>
Date: Wed, 24 Sep 2025 16:50:21 +0200
Subject: [PATCH] xkb: inline SProcXkbGetState()

No need to have whole extra functions for just a few LoC.

Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
---
 xkb/xkb.c     |  7 +++++--
 xkb/xkbSwap.c | 11 +----------
 2 files changed, 6 insertions(+), 12 deletions(-)

diff --git a/xkb/xkb.c b/xkb/xkb.c
index bf1655a10f..b5ffc04ebf 100644
--- a/xkb/xkb.c
+++ b/xkb/xkb.c
@@ -597,11 +597,14 @@ int
 ProcXkbGetState(ClientPtr client)
 {
     REQUEST(xkbGetStateReq);
+    REQUEST_SIZE_MATCH(xkbGetStateReq);
+
+    if (client->swapped)
+        swaps(&stuff->deviceSpec);
+
     DeviceIntPtr dev;
     XkbStateRec *xkb;
 
-    REQUEST_SIZE_MATCH(xkbGetStateReq);
-
     if (!(client->xkbClientFlags & _XkbClientInitialized))
         return BadAccess;
 
diff --git a/xkb/xkbSwap.c b/xkb/xkbSwap.c
index 8f93490de3..f624f4cbd5 100644
--- a/xkb/xkbSwap.c
+++ b/xkb/xkbSwap.c
@@ -118,15 +118,6 @@ SProcXkbSelectEvents(ClientPtr client)
     return ProcXkbSelectEvents(client);
 }
 
-static int _X_COLD
-SProcXkbGetState(ClientPtr client)
-{
-    REQUEST(xkbGetStateReq);
-    REQUEST_SIZE_MATCH(xkbGetStateReq);
-    swaps(&stuff->deviceSpec);
-    return ProcXkbGetState(client);
-}
-
 static int _X_COLD
 SProcXkbLatchLockState(ClientPtr client)
 {
@@ -243,7 +234,7 @@ SProcXkbDispatch(ClientPtr client)
     case X_kbBell:
         return ProcXkbBell(client);
     case X_kbGetState:
-        return SProcXkbGetState(client);
+        return ProcXkbGetState(client);
     case X_kbLatchLockState:
         return SProcXkbLatchLockState(client);
     case X_kbGetControls: