Rémi Bernon (@rbernon) commented about server/protocol.def:
@REPLY obj_handle_t handle; /* next thread handle */ @END + + +/* Setup keyboard auto-repeat */ +(a)REQ(set_keyboard_repeat) + int enable; /* whether to enable auto-repeat */ + int delay; /* auto-repeat delay in ms */ + int period; /* auto-repeat period in ms */ +(a)REPLY + int enable; /* previous state of auto-repeat enable */ + int delay; /* previous auto-repeat delay in ms */ + int period; /* previous auto-repeat period in ms */ If it's not going to be useful in the short term (and as this MR don't use them I would say they aren't) I think it's better to remove the return values and only add them when it will be.
-- https://gitlab.winehq.org/wine/wine/-/merge_requests/5741#note_72478