Xi: inline SProcXIQueryPointer()

Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>

Xi/extinit.c

@@ -432,7 +432,7 @@ SProcIDispatch(ClientPtr client)
             return ProcXGetDeviceProperty(client);
         /* XI 2 */
         case X_XIQueryPointer:
-            return SProcXIQueryPointer(client);
+            return ProcXIQueryPointer(client);
         case X_XIWarpPointer:
             return ProcXIWarpPointer(client);
         case X_XIChangeCursor:

Xi/handlers.h

@@ -70,7 +70,6 @@ int ProcXUngrabDevice(ClientPtr client);
 int ProcXUngrabDeviceKey(ClientPtr client);
 
 int SProcXIGetSelectedEvents(ClientPtr client);
-int SProcXIQueryPointer(ClientPtr client);
 int SProcXISelectEvents(ClientPtr client);
 
 #endif /* _XSERVER_XI_HANDLERS_H */

Xi/xiquerypointer.c

@@ -62,20 +62,17 @@
  *
  */
 
-int _X_COLD
-SProcXIQueryPointer(ClientPtr client)
+int
+ProcXIQueryPointer(ClientPtr client)
 {
     REQUEST(xXIQueryPointerReq);
     REQUEST_SIZE_MATCH(xXIQueryPointerReq);
 
-    swaps(&stuff->deviceid);
-    swapl(&stuff->win);
-    return (ProcXIQueryPointer(client));
-}
+    if (client->swapped) {
+        swaps(&stuff->deviceid);
+        swapl(&stuff->win);
+    }
 
-int
-ProcXIQueryPointer(ClientPtr client)
-{
     int rc;
     DeviceIntPtr pDev, kbd;
     WindowPtr pWin, t;
@@ -83,9 +80,6 @@ ProcXIQueryPointer(ClientPtr client)
     XkbStatePtr state;
     Bool have_xi22 = FALSE;
 
-    REQUEST(xXIQueryPointerReq);
-    REQUEST_SIZE_MATCH(xXIQueryPointerReq);
-
     /* Check if client is compliant with XInput 2.2 or later. Earlier clients
      * do not know about touches, so we must report emulated button presses. 2.2
      * and later clients are aware of touches, so we don't include emulated

test/xi2/protocol-xiquerypointer.c

@@ -133,7 +133,7 @@ request_XIQueryPointer(ClientPtr client, xXIQueryPointerReq * req, int error)
     swaps(&req->deviceid);
     swapl(&req->win);
     swaps(&req->length);
-    rc = SProcXIQueryPointer(&client_request);
+    rc = ProcXIQueryPointer(&client_request);
     assert(rc == error);
 
     if (rc == BadDevice)