From a88bb5f32c80cea72afd76004de53bed67f8cc55 Mon Sep 17 00:00:00 2001 From: Jesse Vincent Date: Wed, 27 Jan 2016 00:07:25 -0800 Subject: [PATCH] We're already checking switch_to_keymap in caller --- KeyboardioFirmware.ino | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/KeyboardioFirmware.ino b/KeyboardioFirmware.ino index 4d4eba26..b4844b1d 100644 --- a/KeyboardioFirmware.ino +++ b/KeyboardioFirmware.ino @@ -133,7 +133,8 @@ void press_key(Key mappedKey) { void handle_keymap_key_event(byte switchState, Key keymapEntry) { // this logic sucks. there is a better way TODO this - if (! (keymapEntry.flags ^ ( MOMENTARY | SWITCH_TO_KEYMAP))) { + + if (! (keymapEntry.flags ^ MOMENTARY ))) { if (key_toggled_on(switchState)) { if ( keymapEntry.rawKey == KEYMAP_NEXT) { temporary_keymap++; @@ -147,7 +148,7 @@ void handle_keymap_key_event(byte switchState, Key keymapEntry) { temporary_keymap = primary_keymap; } - } else if (! (keymapEntry.flags ^ ( SWITCH_TO_KEYMAP ))) { + } else { // switch keymap and stay there if (key_toggled_on(switchState)) { temporary_keymap = primary_keymap = keymapEntry.rawKey;