diff --git a/libraries/Keyboardio-MouseKeys/src/Keyboardio-MouseKeys.cpp b/libraries/Keyboardio-MouseKeys/src/Keyboardio-MouseKeys.cpp index 50ef17a9..c0098279 100644 --- a/libraries/Keyboardio-MouseKeys/src/Keyboardio-MouseKeys.cpp +++ b/libraries/Keyboardio-MouseKeys/src/Keyboardio-MouseKeys.cpp @@ -4,8 +4,8 @@ #include "MouseWrapper.h" #include "KeyboardioFirmware.h" -static void handle_mouse_key_event(Key mappedKey, uint8_t state) { - if (!key_is_pressed(state)) +static void handle_mouse_key_event(Key mappedKey, uint8_t keyState) { + if (!key_is_pressed(keyState)) return; if (mappedKey.rawKey & KEY_MOUSE_UP) { @@ -22,14 +22,14 @@ static void handle_mouse_key_event(Key mappedKey, uint8_t state) { } } -static bool handleMouseKeys(Key mappedKey, byte row, byte col, uint8_t state) { +static bool handleMouseKeys(Key mappedKey, byte row, byte col, uint8_t keyState) { if (! (mappedKey.flags & IS_INTERNAL) && (mappedKey.rawKey == KEY_MOUSE_BTN_L || mappedKey.rawKey == KEY_MOUSE_BTN_M || mappedKey.rawKey == KEY_MOUSE_BTN_R)) { - if (key_toggled_on(state)) { + if (key_toggled_on(keyState)) { MouseWrapper.press_button(mappedKey.rawKey); - } else if (key_toggled_off(state)) { + } else if (key_toggled_off(keyState)) { MouseWrapper.release_button(mappedKey.rawKey); } return true; @@ -39,8 +39,8 @@ static bool handleMouseKeys(Key mappedKey, byte row, byte col, uint8_t state) { return false; if (!(mappedKey.rawKey & KEY_MOUSE_WARP)) { - handle_mouse_key_event(mappedKey, state); - } else if (key_toggled_on(state)) { + handle_mouse_key_event(mappedKey, keyState); + } else if (key_toggled_on(keyState)) { if (mappedKey.rawKey & KEY_MOUSE_WARP && mappedKey.flags & IS_MOUSE_KEY) { // we don't pass in the left and up values because those are the // default, "no-op" conditionals diff --git a/src/Model01.cpp b/src/Model01.cpp index 446dd31c..f4301dce 100644 --- a/src/Model01.cpp +++ b/src/Model01.cpp @@ -130,14 +130,14 @@ void Model01::act_on_matrix_scan() { uint8_t keynum = (row*8)+(col); - uint8_t state = (bitRead(previousLeftHandState.all, keynum) << 0) | + uint8_t keyState = (bitRead(previousLeftHandState.all, keynum) << 0) | (bitRead(leftHandState.all, keynum) << 1); - handle_key_event(Key_NoKey, row, 7-col, state); + handle_key_event(Key_NoKey, row, 7-col, keyState); - state = (bitRead(previousRightHandState.all, keynum) << 0) | + keyState = (bitRead(previousRightHandState.all, keynum) << 0) | (bitRead(rightHandState.all, keynum) << 1); - handle_key_event(Key_NoKey, row, (15- col), state); + handle_key_event(Key_NoKey, row, (15- col), keyState); } } } diff --git a/src/TestMode.cpp b/src/TestMode.cpp index 2a13170a..1d1c34ad 100644 --- a/src/TestMode.cpp +++ b/src/TestMode.cpp @@ -64,7 +64,7 @@ void TestMode_::TestMatrix () { void TestMode_::setup() { eventHandlers[0] = handle_key_event_test; } -bool handle_key_event_test(Key mappedKey, byte row, byte col, uint8_t state) { +bool handle_key_event_test(Key mappedKey, byte row, byte col, uint8_t keyState) { Serial.write(row); return false; } diff --git a/src/TestMode.h b/src/TestMode.h index 9189b723..c25f6f3e 100644 --- a/src/TestMode.h +++ b/src/TestMode.h @@ -15,4 +15,4 @@ class TestMode_ { extern TestMode_ TestMode; -bool handle_key_event_test(Key mappedKey, byte row, byte col, uint8_t state); +bool handle_key_event_test(Key mappedKey, byte row, byte col, uint8_t keyState); diff --git a/src/hooks.h b/src/hooks.h index 57b65568..2499174b 100644 --- a/src/hooks.h +++ b/src/hooks.h @@ -5,7 +5,7 @@ #define HOOK_MAX 64 -typedef bool (*custom_handler_t)(Key mappedKey, byte row, byte col, uint8_t state); +typedef bool (*custom_handler_t)(Key mappedKey, byte row, byte col, uint8_t keyState); extern custom_handler_t eventHandlers[HOOK_MAX]; void event_handler_hook_add (custom_handler_t hook); diff --git a/src/key_events.cpp b/src/key_events.cpp index 72e14485..ba345d1d 100644 --- a/src/key_events.cpp +++ b/src/key_events.cpp @@ -1,7 +1,7 @@ #include "key_events.h" -void handle_synthetic_key_event(Key mappedKey, uint8_t state) { - if (key_toggled_on(state)) { +void handle_synthetic_key_event(Key mappedKey, uint8_t keyState) { + if (key_toggled_on(keyState)) { if (mappedKey.flags & IS_CONSUMER) { ConsumerControl.press(mappedKey.rawKey); } else if (mappedKey.flags & IS_INTERNAL) { @@ -29,32 +29,32 @@ Key lookup_key(byte keymap, byte row, byte col) { return mappedKey; } -void handle_key_event(Key mappedKey, byte row, byte col, uint8_t state) { - if (!(state & INJECTED)) { +void handle_key_event(Key mappedKey, byte row, byte col, uint8_t keyState) { + if (!(keyState & INJECTED)) { mappedKey = lookup_key(temporary_keymap, row, col); } for (byte i = 0; eventHandlers[i] != NULL && i < HOOK_MAX; i++) { custom_handler_t handler = eventHandlers[i]; - if ((*handler)(mappedKey, row, col, state)) + if ((*handler)(mappedKey, row, col, keyState)) return; } } -bool handle_key_event_default(Key mappedKey, byte row, byte col, uint8_t state) { +bool handle_key_event_default(Key mappedKey, byte row, byte col, uint8_t keyState) { //for every newly pressed button, figure out what logical key it is and send a key down event // for every newly released button, figure out what logical key it is and send a key up event Key baseKey = Key_NoKey; - if (!(state & INJECTED)) { + if (!(keyState & INJECTED)) { baseKey = lookup_key(primary_keymap, row, col); } if ((baseKey.flags & SWITCH_TO_KEYMAP || baseKey.flags & SWITCH_TO_KEYMAP_MOMENTARY)) { - handle_keymap_key_event(baseKey, state); + handle_keymap_key_event(baseKey, keyState); } else if (mappedKey.flags & SYNTHETIC) { - handle_synthetic_key_event( mappedKey, state); - } else if (key_is_pressed(state)) { + handle_synthetic_key_event( mappedKey, keyState); + } else if (key_is_pressed(keyState)) { press_key(mappedKey); } return true; @@ -80,9 +80,9 @@ void press_key(Key mappedKey) { } -void handle_keymap_key_event(Key keymapEntry, uint8_t state) { +void handle_keymap_key_event(Key keymapEntry, uint8_t keyState) { if (keymapEntry.flags & SWITCH_TO_KEYMAP_MOMENTARY ) { - if (key_toggled_on(state)) { + if (key_toggled_on(keyState)) { if ( keymapEntry.rawKey == KEYMAP_NEXT) { temporary_keymap++; } else if ( keymapEntry.rawKey == KEYMAP_PREVIOUS) { @@ -91,12 +91,12 @@ void handle_keymap_key_event(Key keymapEntry, uint8_t state) { temporary_keymap = keymapEntry.rawKey; } } - if (key_toggled_off(state)) { + if (key_toggled_off(keyState)) { temporary_keymap = primary_keymap; } // switch keymap and stay there - } else if (key_toggled_on(state)) { + } else if (key_toggled_on(keyState)) { temporary_keymap = primary_keymap = keymapEntry.rawKey; Storage.save_primary_keymap(primary_keymap); } diff --git a/src/key_events.h b/src/key_events.h index b46d6382..d7128a1b 100644 --- a/src/key_events.h +++ b/src/key_events.h @@ -43,12 +43,12 @@ extern const Key keymaps[][ROWS][COLS]; * currentState may be flagged INJECTED, which signals that the event was * injected, and is not a direct result of a keypress, coming from the scanner. */ -void handle_key_event(Key mappedKey, byte row, byte col, uint8_t state); +void handle_key_event(Key mappedKey, byte row, byte col, uint8_t keyState); // Internal use -void handle_synthetic_key_event( Key mappedKey, uint8_t state); +void handle_synthetic_key_event( Key mappedKey, uint8_t keyState); void press_key(Key mappedKey); -void handle_keymap_key_event(Key keymapEntry, uint8_t state); -bool handle_key_event_default(Key mappedKey, byte row, byte col, uint8_t state); +void handle_keymap_key_event(Key keymapEntry, uint8_t keyState); +bool handle_key_event_default(Key mappedKey, byte row, byte col, uint8_t keyState); Key lookup_key(byte keymap, byte row, byte col); diff --git a/src/keyswitch_state.h b/src/keyswitch_state.h index 80eab13a..2e13714f 100644 --- a/src/keyswitch_state.h +++ b/src/keyswitch_state.h @@ -7,7 +7,7 @@ #define IS_PRESSED B00000010 #define WAS_PRESSED B00000001 -#define key_was_pressed(state) (state & WAS_PRESSED) -#define key_is_pressed(state) (state & IS_PRESSED) -#define key_toggled_on(state) (key_is_pressed(state) && ! key_was_pressed(state)) -#define key_toggled_off(state) (key_was_pressed(state) && ! key_is_pressed(state)) +#define key_was_pressed(keyState) (keyState & WAS_PRESSED) +#define key_is_pressed(keyState) (keyState & IS_PRESSED) +#define key_toggled_on(keyState) (key_is_pressed(keyState) && ! key_was_pressed(keyState)) +#define key_toggled_off(keyState) (key_was_pressed(keyState) && ! key_is_pressed(keyState))