aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDrew DeVault <sir@cmpwn.com>2015-08-19 20:52:52 -0400
committerDrew DeVault <sir@cmpwn.com>2015-08-19 20:52:52 -0400
commit1d8591d9026c19f5c308bed5961341b00aef69ca (patch)
tree1bdb05a71b0128b014901af8d60608869ec58000
parent84a778b688320ca535cee29f1b6c580ed7f36715 (diff)
Improve key handling somewhat
XKB is fucking bullshit
-rw-r--r--sway/handlers.c30
1 files changed, 25 insertions, 5 deletions
diff --git a/sway/handlers.c b/sway/handlers.c
index c789662e..18f1d13c 100644
--- a/sway/handlers.c
+++ b/sway/handlers.c
@@ -316,6 +316,14 @@ static void handle_view_state_request(wlc_handle view, enum wlc_view_state_bit s
static bool handle_key(wlc_handle view, uint32_t time, const struct wlc_modifiers *modifiers,
uint32_t key, uint32_t sym, enum wlc_key_state state) {
+
+ static const uint32_t modifier_syms[] = {
+ XKB_KEY_Shift_L, XKB_KEY_Shift_R, XKB_KEY_Control_L, XKB_KEY_Control_R,
+ XKB_KEY_Caps_Lock, XKB_KEY_Shift_Lock, XKB_KEY_Meta_L, XKB_KEY_Meta_R,
+ XKB_KEY_Alt_L, XKB_KEY_Alt_R, XKB_KEY_Super_L, XKB_KEY_Super_R,
+ XKB_KEY_Hyper_L, XKB_KEY_Hyper_R
+ };
+
if (locked_view_focus && state == WLC_KEY_STATE_PRESSED) {
return false;
}
@@ -327,12 +335,28 @@ static bool handle_key(wlc_handle view, uint32_t time, const struct wlc_modifier
}
struct sway_mode *mode = config->current_mode;
+
+ if (!isalnum(sym)) {
+ // God fucking dammit
+ return false;
+ }
+
// Lowercase if necessary
sym = tolower(sym);
int i;
+ bool mod = false;
- for (i = 0; i < KEY_CACHE_SIZE; ++i) {
+ for (i = 0; i < sizeof(modifier_syms) / sizeof(uint32_t); ++i) {
+ if (modifier_syms[i] == sym) {
+ mod = true;
+ break;
+ }
+ }
+
+ int total = 0;
+ for (i = 0; i < KEY_CACHE_SIZE && !mod; ++i) {
+ total += keys_pressed[i] != 0;
if (state == WLC_KEY_STATE_PRESSED && keys_pressed[i] == 0) {
keys_pressed[i] = sym;
break;
@@ -341,10 +365,6 @@ static bool handle_key(wlc_handle view, uint32_t time, const struct wlc_modifier
break;
}
}
- if (i == KEY_CACHE_SIZE) {
- sway_log(L_DEBUG, "Key buffer full!");
- return false;
- }
// TODO: reminder to check conflicts with mod+q+a versus mod+q
for (i = 0; i < mode->bindings->length; ++i) {