Xi: drop now obsolete swap procs

Lots of SProc's have become no-ops, just calling the actual Proc's,
so we can get rid of them entirely.

Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1639>
This commit is contained in:
Enrico Weigelt, metux IT consult 2024-08-06 15:21:48 +02:00 committed by Marge Bot
parent dc9a998b96
commit 190346663b
38 changed files with 20 additions and 282 deletions

View File

@ -69,20 +69,6 @@ SOFTWARE.
* This procedure changes the keyboard device. * This procedure changes the keyboard device.
* *
*/ */
int _X_COLD
SProcXChangeKeyboardDevice(ClientPtr client)
{
return (ProcXChangeKeyboardDevice(client));
}
/***********************************************************************
*
* This procedure is invoked to swap the request bytes if the server and
* client have a different byte order.
*
*/
int int
ProcXChangeKeyboardDevice(ClientPtr client) ProcXChangeKeyboardDevice(ClientPtr client)
{ {

View File

@ -30,9 +30,6 @@ OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
#ifndef CHGKBD_H #ifndef CHGKBD_H
#define CHGKBD_H 1 #define CHGKBD_H 1
int SProcXChangeKeyboardDevice(ClientPtr /* client */
);
int ProcXChangeKeyboardDevice(ClientPtr /* client */ int ProcXChangeKeyboardDevice(ClientPtr /* client */
); );

View File

@ -65,19 +65,6 @@ SOFTWARE.
#include "chgptr.h" #include "chgptr.h"
/***********************************************************************
*
* This procedure is invoked to swap the request bytes if the server and
* client have a different byte order.
*
*/
int _X_COLD
SProcXChangePointerDevice(ClientPtr client)
{
return (ProcXChangePointerDevice(client));
}
/*********************************************************************** /***********************************************************************
* *
* This procedure changes the device used as the X pointer. * This procedure changes the device used as the X pointer.

View File

@ -30,9 +30,6 @@ OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
#ifndef CHGPTR_H #ifndef CHGPTR_H
#define CHGPTR_H 1 #define CHGPTR_H 1
int SProcXChangePointerDevice(ClientPtr /* client */
);
int ProcXChangePointerDevice(ClientPtr /* client */ int ProcXChangePointerDevice(ClientPtr /* client */
); );

View File

@ -62,18 +62,6 @@ SOFTWARE.
#include "closedev.h" #include "closedev.h"
/***********************************************************************
*
* This procedure closes an input device.
*
*/
int _X_COLD
SProcXCloseDevice(ClientPtr client)
{
return (ProcXCloseDevice(client));
}
/*********************************************************************** /***********************************************************************
* *
* Clear out event selections and passive grabs from a window for the * Clear out event selections and passive grabs from a window for the

View File

@ -30,9 +30,6 @@ OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
#ifndef CLOSEDEV_H #ifndef CLOSEDEV_H
#define CLOSEDEV_H 1 #define CLOSEDEV_H 1
int SProcXCloseDevice(ClientPtr /* client */
);
int ProcXCloseDevice(ClientPtr /* client */ int ProcXCloseDevice(ClientPtr /* client */
); );

View File

@ -59,19 +59,6 @@ SOFTWARE.
#include "devbell.h" #include "devbell.h"
/***********************************************************************
*
* This procedure is invoked to swap the request bytes if the server and
* client have a different byte order.
*
*/
int _X_COLD
SProcXDeviceBell(ClientPtr client)
{
return (ProcXDeviceBell(client));
}
/*********************************************************************** /***********************************************************************
* *
* This procedure rings a bell on an extension device. * This procedure rings a bell on an extension device.

View File

@ -30,9 +30,6 @@ OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
#ifndef DEVBELL_H #ifndef DEVBELL_H
#define DEVBELL_H 1 #define DEVBELL_H 1
int SProcXDeviceBell(ClientPtr /* client */
);
int ProcXDeviceBell(ClientPtr /* client */ int ProcXDeviceBell(ClientPtr /* client */
); );

View File

@ -252,19 +252,19 @@ static int (*ProcIVector[]) (ClientPtr) = {
/* For swapped clients */ /* For swapped clients */
static int (*SProcIVector[]) (ClientPtr) = { static int (*SProcIVector[]) (ClientPtr) = {
NULL, /* 0 */ NULL, /* 0 */
SProcXGetExtensionVersion, /* 1 */ SProcXGetExtensionVersion, /* 1 */
SProcXListInputDevices, /* 2 */ ProcXListInputDevices, /* 2 */
SProcXOpenDevice, /* 3 */ ProcXOpenDevice, /* 3 */
SProcXCloseDevice, /* 4 */ ProcXCloseDevice, /* 4 */
SProcXSetDeviceMode, /* 5 */ ProcXSetDeviceMode, /* 5 */
SProcXSelectExtensionEvent, /* 6 */ SProcXSelectExtensionEvent, /* 6 */
SProcXGetSelectedExtensionEvents, /* 7 */ SProcXGetSelectedExtensionEvents, /* 7 */
SProcXChangeDeviceDontPropagateList, /* 8 */ SProcXChangeDeviceDontPropagateList, /* 8 */
SProcXGetDeviceDontPropagateList, /* 9 */ SProcXGetDeviceDontPropagateList, /* 9 */
SProcXGetDeviceMotionEvents, /* 10 */ SProcXGetDeviceMotionEvents, /* 10 */
SProcXChangeKeyboardDevice, /* 11 */ ProcXChangeKeyboardDevice, /* 11 */
SProcXChangePointerDevice, /* 12 */ ProcXChangePointerDevice, /* 12 */
SProcXGrabDevice, /* 13 */ SProcXGrabDevice, /* 13 */
SProcXUngrabDevice, /* 14 */ SProcXUngrabDevice, /* 14 */
SProcXGrabDeviceKey, /* 15 */ SProcXGrabDeviceKey, /* 15 */
@ -272,30 +272,30 @@ static int (*SProcIVector[]) (ClientPtr) = {
SProcXGrabDeviceButton, /* 17 */ SProcXGrabDeviceButton, /* 17 */
SProcXUngrabDeviceButton, /* 18 */ SProcXUngrabDeviceButton, /* 18 */
SProcXAllowDeviceEvents, /* 19 */ SProcXAllowDeviceEvents, /* 19 */
SProcXGetDeviceFocus, /* 20 */ ProcXGetDeviceFocus, /* 20 */
SProcXSetDeviceFocus, /* 21 */ SProcXSetDeviceFocus, /* 21 */
SProcXGetFeedbackControl, /* 22 */ ProcXGetFeedbackControl, /* 22 */
SProcXChangeFeedbackControl, /* 23 */ SProcXChangeFeedbackControl, /* 23 */
SProcXGetDeviceKeyMapping, /* 24 */ ProcXGetDeviceKeyMapping, /* 24 */
SProcXChangeDeviceKeyMapping, /* 25 */ SProcXChangeDeviceKeyMapping, /* 25 */
SProcXGetDeviceModifierMapping, /* 26 */ ProcXGetDeviceModifierMapping, /* 26 */
SProcXSetDeviceModifierMapping, /* 27 */ ProcXSetDeviceModifierMapping, /* 27 */
SProcXGetDeviceButtonMapping, /* 28 */ ProcXGetDeviceButtonMapping, /* 28 */
SProcXSetDeviceButtonMapping, /* 29 */ ProcXSetDeviceButtonMapping, /* 29 */
SProcXQueryDeviceState, /* 30 */ ProcXQueryDeviceState, /* 30 */
SProcXSendExtensionEvent, /* 31 */ SProcXSendExtensionEvent, /* 31 */
SProcXDeviceBell, /* 32 */ ProcXDeviceBell, /* 32 */
SProcXSetDeviceValuators, /* 33 */ ProcXSetDeviceValuators, /* 33 */
SProcXGetDeviceControl, /* 34 */ SProcXGetDeviceControl, /* 34 */
SProcXChangeDeviceControl, /* 35 */ SProcXChangeDeviceControl, /* 35 */
SProcXListDeviceProperties, /* 36 */ ProcXListDeviceProperties, /* 36 */
SProcXChangeDeviceProperty, /* 37 */ SProcXChangeDeviceProperty, /* 37 */
SProcXDeleteDeviceProperty, /* 38 */ SProcXDeleteDeviceProperty, /* 38 */
SProcXGetDeviceProperty, /* 39 */ SProcXGetDeviceProperty, /* 39 */
SProcXIQueryPointer, /* 40 */ SProcXIQueryPointer, /* 40 */
SProcXIWarpPointer, /* 41 */ SProcXIWarpPointer, /* 41 */
SProcXIChangeCursor, /* 42 */ SProcXIChangeCursor, /* 42 */
SProcXIChangeHierarchy, /* 43 */ ProcXIChangeHierarchy, /* 43 */
SProcXISetClientPointer, /* 44 */ SProcXISetClientPointer, /* 44 */
SProcXIGetClientPointer, /* 45 */ SProcXIGetClientPointer, /* 45 */
SProcXISelectEvents, /* 46 */ SProcXISelectEvents, /* 46 */

View File

@ -59,18 +59,6 @@ SOFTWARE.
#include "getbmap.h" #include "getbmap.h"
/***********************************************************************
*
* This procedure gets the button mapping for the specified device.
*
*/
int _X_COLD
SProcXGetDeviceButtonMapping(ClientPtr client)
{
return (ProcXGetDeviceButtonMapping(client));
}
/*********************************************************************** /***********************************************************************
* *
* This procedure gets the button mapping for the specified device. * This procedure gets the button mapping for the specified device.

View File

@ -30,9 +30,6 @@ OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
#ifndef GETBMAP_H #ifndef GETBMAP_H
#define GETBMAP_H 1 #define GETBMAP_H 1
int SProcXGetDeviceButtonMapping(ClientPtr /* client */
);
int ProcXGetDeviceButtonMapping(ClientPtr /* client */ int ProcXGetDeviceButtonMapping(ClientPtr /* client */
); );

View File

@ -59,19 +59,6 @@ SOFTWARE.
#include "getfctl.h" #include "getfctl.h"
/***********************************************************************
*
* This procedure gets the control attributes for an extension device,
* for clients on machines with a different byte ordering than the server.
*
*/
int _X_COLD
SProcXGetFeedbackControl(ClientPtr client)
{
return (ProcXGetFeedbackControl(client));
}
/*********************************************************************** /***********************************************************************
* *
* This procedure copies KbdFeedbackClass data, swapping if necessary. * This procedure copies KbdFeedbackClass data, swapping if necessary.

View File

@ -30,9 +30,6 @@ OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
#ifndef GETFCTL_H #ifndef GETFCTL_H
#define GETFCTL_H 1 #define GETFCTL_H 1
int SProcXGetFeedbackControl(ClientPtr /* client */
);
int ProcXGetFeedbackControl(ClientPtr /* client */ int ProcXGetFeedbackControl(ClientPtr /* client */
); );

View File

@ -60,18 +60,6 @@ SOFTWARE.
#include "getfocus.h" #include "getfocus.h"
/***********************************************************************
*
* This procedure gets the focus for a device.
*
*/
int _X_COLD
SProcXGetDeviceFocus(ClientPtr client)
{
return (ProcXGetDeviceFocus(client));
}
/*********************************************************************** /***********************************************************************
* *
* This procedure gets the focus for a device. * This procedure gets the focus for a device.

View File

@ -30,9 +30,6 @@ OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
#ifndef GETFOCUS_H #ifndef GETFOCUS_H
#define GETFOCUS_H 1 #define GETFOCUS_H 1
int SProcXGetDeviceFocus(ClientPtr /* client */
);
int ProcXGetDeviceFocus(ClientPtr /* client */ int ProcXGetDeviceFocus(ClientPtr /* client */
); );

View File

@ -62,19 +62,6 @@ SOFTWARE.
#include "getkmap.h" #include "getkmap.h"
/***********************************************************************
*
* This procedure gets the key mapping for an extension device,
* for clients on machines with a different byte ordering than the server.
*
*/
int _X_COLD
SProcXGetDeviceKeyMapping(ClientPtr client)
{
return (ProcXGetDeviceKeyMapping(client));
}
/*********************************************************************** /***********************************************************************
* *
* Get the device key mapping. * Get the device key mapping.

View File

@ -30,9 +30,6 @@ OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
#ifndef GETKMAP_H #ifndef GETKMAP_H
#define GETKMAP_H 1 #define GETKMAP_H 1
int SProcXGetDeviceKeyMapping(ClientPtr /* client */
);
int ProcXGetDeviceKeyMapping(ClientPtr /* client */ int ProcXGetDeviceKeyMapping(ClientPtr /* client */
); );

View File

@ -59,19 +59,6 @@ SOFTWARE.
#include "getmmap.h" #include "getmmap.h"
/***********************************************************************
*
* This procedure gets the modifier mapping for an extension device,
* for clients on machines with a different byte ordering than the server.
*
*/
int _X_COLD
SProcXGetDeviceModifierMapping(ClientPtr client)
{
return (ProcXGetDeviceModifierMapping(client));
}
/*********************************************************************** /***********************************************************************
* *
* Get the device Modifier mapping. * Get the device Modifier mapping.

View File

@ -30,9 +30,6 @@ OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
#ifndef GETMMAP_H #ifndef GETMMAP_H
#define GETMMAP_H 1 #define GETMMAP_H 1
int SProcXGetDeviceModifierMapping(ClientPtr /* client */
);
int ProcXGetDeviceModifierMapping(ClientPtr /* client */ int ProcXGetDeviceModifierMapping(ClientPtr /* client */
); );

View File

@ -77,6 +77,7 @@ SProcXGrabDevice(ClientPtr client)
{ {
REQUEST(xGrabDeviceReq); REQUEST(xGrabDeviceReq);
REQUEST_AT_LEAST_SIZE(xGrabDeviceReq); REQUEST_AT_LEAST_SIZE(xGrabDeviceReq);
swapl(&stuff->grabWindow); swapl(&stuff->grabWindow);
swapl(&stuff->time); swapl(&stuff->time);
swaps(&stuff->event_count); swaps(&stuff->event_count);

View File

@ -68,18 +68,6 @@ SOFTWARE.
#include "xkbstr.h" #include "xkbstr.h"
#include "listdev.h" #include "listdev.h"
/***********************************************************************
*
* This procedure lists the input devices available to the server.
*
*/
int _X_COLD
SProcXListInputDevices(ClientPtr client)
{
return (ProcXListInputDevices(client));
}
/*********************************************************************** /***********************************************************************
* *
* This procedure calculates the size of the information to be returned * This procedure calculates the size of the information to be returned

View File

@ -32,9 +32,6 @@ OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
#define VPC 20 /* Max # valuators per chunk */ #define VPC 20 /* Max # valuators per chunk */
int SProcXListInputDevices(ClientPtr /* client */
);
int ProcXListInputDevices(ClientPtr /* client */ int ProcXListInputDevices(ClientPtr /* client */
); );

View File

@ -64,19 +64,6 @@ SOFTWARE.
extern CARD8 event_base[]; extern CARD8 event_base[];
/***********************************************************************
*
* This procedure swaps the request if the server and client have different
* byte orderings.
*
*/
int _X_COLD
SProcXOpenDevice(ClientPtr client)
{
return (ProcXOpenDevice(client));
}
/*********************************************************************** /***********************************************************************
* *
* This procedure causes the server to open an input device. * This procedure causes the server to open an input device.

View File

@ -30,9 +30,6 @@ OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
#ifndef OPENDEV_H #ifndef OPENDEV_H
#define OPENDEV_H 1 #define OPENDEV_H 1
int SProcXOpenDevice(ClientPtr /* client */
);
int ProcXOpenDevice(ClientPtr /* client */ int ProcXOpenDevice(ClientPtr /* client */
); );

View File

@ -47,18 +47,6 @@ from The Open Group.
#include "xkbstr.h" #include "xkbstr.h"
#include "queryst.h" #include "queryst.h"
/***********************************************************************
*
* This procedure allows a client to query the state of a device.
*
*/
int _X_COLD
SProcXQueryDeviceState(ClientPtr client)
{
return (ProcXQueryDeviceState(client));
}
/*********************************************************************** /***********************************************************************
* *
* This procedure allows frozen events to be routed. * This procedure allows frozen events to be routed.

View File

@ -30,9 +30,6 @@ OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
#ifndef QUERYST_H #ifndef QUERYST_H
#define QUERYST_H 1 #define QUERYST_H 1
int SProcXQueryDeviceState(ClientPtr /* client */
);
int ProcXQueryDeviceState(ClientPtr /* client */ int ProcXQueryDeviceState(ClientPtr /* client */
); );

View File

@ -62,18 +62,6 @@ SOFTWARE.
#include "exglobals.h" #include "exglobals.h"
#include "setbmap.h" #include "setbmap.h"
/***********************************************************************
*
* This procedure changes the button mapping.
*
*/
int _X_COLD
SProcXSetDeviceButtonMapping(ClientPtr client)
{
return (ProcXSetDeviceButtonMapping(client));
}
/*********************************************************************** /***********************************************************************
* *
* This procedure lists the input devices available to the server. * This procedure lists the input devices available to the server.

View File

@ -30,9 +30,6 @@ OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
#ifndef SETBMAP_H #ifndef SETBMAP_H
#define SETBMAP_H 1 #define SETBMAP_H 1
int SProcXSetDeviceButtonMapping(ClientPtr /* client */
);
int ProcXSetDeviceButtonMapping(ClientPtr /* client */ int ProcXSetDeviceButtonMapping(ClientPtr /* client */
); );

View File

@ -62,18 +62,6 @@ SOFTWARE.
#include "exglobals.h" #include "exglobals.h"
#include "setdval.h" #include "setdval.h"
/***********************************************************************
*
* Handle a request from a client with a different byte order.
*
*/
int _X_COLD
SProcXSetDeviceValuators(ClientPtr client)
{
return (ProcXSetDeviceValuators(client));
}
/*********************************************************************** /***********************************************************************
* *
* This procedure sets the value of valuators on an extension input device. * This procedure sets the value of valuators on an extension input device.

View File

@ -30,9 +30,6 @@ OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
#ifndef SETDVAL_H #ifndef SETDVAL_H
#define SETDVAL_H 1 #define SETDVAL_H 1
int SProcXSetDeviceValuators(ClientPtr /* client */
);
int ProcXSetDeviceValuators(ClientPtr /* client */ int ProcXSetDeviceValuators(ClientPtr /* client */
); );

View File

@ -63,19 +63,6 @@ SOFTWARE.
#include "exglobals.h" #include "exglobals.h"
#include "setmmap.h" #include "setmmap.h"
/***********************************************************************
*
* This procedure sets the modifier mapping for an extension device,
* for clients on machines with a different byte ordering than the server.
*
*/
int _X_COLD
SProcXSetDeviceModifierMapping(ClientPtr client)
{
return (ProcXSetDeviceModifierMapping(client));
}
/*********************************************************************** /***********************************************************************
* *
* Set the device Modifier mapping. * Set the device Modifier mapping.

View File

@ -30,9 +30,6 @@ OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
#ifndef SETMMAP_H #ifndef SETMMAP_H
#define SETMMAP_H 1 #define SETMMAP_H 1
int SProcXSetDeviceModifierMapping(ClientPtr /* client */
);
int ProcXSetDeviceModifierMapping(ClientPtr /* client */ int ProcXSetDeviceModifierMapping(ClientPtr /* client */
); );

View File

@ -62,18 +62,6 @@ SOFTWARE.
#include "exglobals.h" #include "exglobals.h"
#include "setmode.h" #include "setmode.h"
/***********************************************************************
*
* Handle a request from a client with a different byte order.
*
*/
int _X_COLD
SProcXSetDeviceMode(ClientPtr client)
{
return (ProcXSetDeviceMode(client));
}
/*********************************************************************** /***********************************************************************
* *
* This procedure sets the mode of a device. * This procedure sets the mode of a device.

View File

@ -30,9 +30,6 @@ OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
#ifndef SETMODE_H #ifndef SETMODE_H
#define SETMODE_H 1 #define SETMODE_H 1
int SProcXSetDeviceMode(ClientPtr /* client */
);
int ProcXSetDeviceMode(ClientPtr /* client */ int ProcXSetDeviceMode(ClientPtr /* client */
); );

View File

@ -128,12 +128,6 @@ XISendDeviceHierarchyEvent(int flags[MAXDEVICES])
* *
*/ */
int _X_COLD
SProcXIChangeHierarchy(ClientPtr client)
{
return (ProcXIChangeHierarchy(client));
}
static int static int
add_master(ClientPtr client, xXIAddMasterInfo * c, int flags[MAXDEVICES]) add_master(ClientPtr client, xXIAddMasterInfo * c, int flags[MAXDEVICES])
{ {

View File

@ -36,7 +36,6 @@
#ifndef CHDEVHIER_H #ifndef CHDEVHIER_H
#define CHDEVHIER_H 1 #define CHDEVHIER_H 1
int SProcXIChangeHierarchy(ClientPtr /* client */ );
int ProcXIChangeHierarchy(ClientPtr /* client */ ); int ProcXIChangeHierarchy(ClientPtr /* client */ );
void XISendDeviceHierarchyEvent(int flags[MAXDEVICES]); void XISendDeviceHierarchyEvent(int flags[MAXDEVICES]);

View File

@ -1013,12 +1013,6 @@ ProcXGetDeviceProperty(ClientPtr client)
return Success; return Success;
} }
int _X_COLD
SProcXListDeviceProperties(ClientPtr client)
{
return (ProcXListDeviceProperties(client));
}
int _X_COLD int _X_COLD
SProcXChangeDeviceProperty(ClientPtr client) SProcXChangeDeviceProperty(ClientPtr client)
{ {

View File

@ -36,7 +36,6 @@ int ProcXDeleteDeviceProperty(ClientPtr client);
int ProcXGetDeviceProperty(ClientPtr client); int ProcXGetDeviceProperty(ClientPtr client);
/* request swapping */ /* request swapping */
int SProcXListDeviceProperties(ClientPtr client);
int SProcXChangeDeviceProperty(ClientPtr client); int SProcXChangeDeviceProperty(ClientPtr client);
int SProcXDeleteDeviceProperty(ClientPtr client); int SProcXDeleteDeviceProperty(ClientPtr client);
int SProcXGetDeviceProperty(ClientPtr client); int SProcXGetDeviceProperty(ClientPtr client);