Merge pull request #3309 from orestisf1993/detectable-autorepeat

Enable detectable autorepeat
This commit is contained in:
Ingo Bürk 2018-06-20 08:38:46 +02:00 committed by GitHub
commit 6c34063f95
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 22 additions and 10 deletions

View File

@ -642,7 +642,15 @@ int main(int argc, char *argv[]) {
* XCB_XKB_PER_CLIENT_FLAG_LOOKUP_STATE_WHEN_GRABBED, will lead to the * XCB_XKB_PER_CLIENT_FLAG_LOOKUP_STATE_WHEN_GRABBED, will lead to the
* X server sending us the full XKB state in KeyPress and KeyRelease: * X server sending us the full XKB state in KeyPress and KeyRelease:
* https://cgit.freedesktop.org/xorg/xserver/tree/xkb/xkbEvents.c?h=xorg-server-1.20.0#n927 * https://cgit.freedesktop.org/xorg/xserver/tree/xkb/xkbEvents.c?h=xorg-server-1.20.0#n927
*
* XCB_XKB_PER_CLIENT_FLAG_DETECTABLE_AUTO_REPEAT enable detectable autorepeat:
* https://www.x.org/releases/current/doc/kbproto/xkbproto.html#Detectable_Autorepeat
* This affects bindings using the --release flag: instead of getting multiple KeyRelease
* events we get only one event when the key is physically released by the user.
*/ */
const uint32_t mask = XCB_XKB_PER_CLIENT_FLAG_GRABS_USE_XKB_STATE |
XCB_XKB_PER_CLIENT_FLAG_LOOKUP_STATE_WHEN_GRABBED |
XCB_XKB_PER_CLIENT_FLAG_DETECTABLE_AUTO_REPEAT;
xcb_xkb_per_client_flags_reply_t *pcf_reply; xcb_xkb_per_client_flags_reply_t *pcf_reply;
/* The last three parameters are unset because they are only relevant /* The last three parameters are unset because they are only relevant
* when using a feature called automatic reset of boolean controls: * when using a feature called automatic reset of boolean controls:
@ -653,20 +661,24 @@ int main(int argc, char *argv[]) {
xcb_xkb_per_client_flags( xcb_xkb_per_client_flags(
conn, conn,
XCB_XKB_ID_USE_CORE_KBD, XCB_XKB_ID_USE_CORE_KBD,
XCB_XKB_PER_CLIENT_FLAG_GRABS_USE_XKB_STATE | XCB_XKB_PER_CLIENT_FLAG_LOOKUP_STATE_WHEN_GRABBED, mask,
XCB_XKB_PER_CLIENT_FLAG_GRABS_USE_XKB_STATE | XCB_XKB_PER_CLIENT_FLAG_LOOKUP_STATE_WHEN_GRABBED, mask,
0 /* uint32_t ctrlsToChange */, 0 /* uint32_t ctrlsToChange */,
0 /* uint32_t autoCtrls */, 0 /* uint32_t autoCtrls */,
0 /* uint32_t autoCtrlsValues */), 0 /* uint32_t autoCtrlsValues */),
NULL); NULL);
if (pcf_reply == NULL ||
!(pcf_reply->value & XCB_XKB_PER_CLIENT_FLAG_GRABS_USE_XKB_STATE)) { #define PCF_REPLY_ERROR(_value) \
ELOG("Could not set XCB_XKB_PER_CLIENT_FLAG_GRABS_USE_XKB_STATE\n"); do { \
} if (pcf_reply == NULL || !(pcf_reply->value & (_value))) { \
if (pcf_reply == NULL || ELOG("Could not set " #_value "\n"); \
!(pcf_reply->value & XCB_XKB_PER_CLIENT_FLAG_LOOKUP_STATE_WHEN_GRABBED)) { } \
ELOG("Could not set XCB_XKB_PER_CLIENT_FLAG_LOOKUP_STATE_WHEN_GRABBED\n"); } while (0)
}
PCF_REPLY_ERROR(XCB_XKB_PER_CLIENT_FLAG_GRABS_USE_XKB_STATE);
PCF_REPLY_ERROR(XCB_XKB_PER_CLIENT_FLAG_LOOKUP_STATE_WHEN_GRABBED);
PCF_REPLY_ERROR(XCB_XKB_PER_CLIENT_FLAG_DETECTABLE_AUTO_REPEAT);
free(pcf_reply); free(pcf_reply);
xkb_base = extreply->first_event; xkb_base = extreply->first_event;
} }