diff options
author | Pete Sevander <[email protected]> | 2023-05-10 18:59:52 +0300 |
---|---|---|
committer | GitHub <[email protected]> | 2023-05-10 17:59:52 +0200 |
commit | 8a332e6f0105d2db9239e3c3f997bae754522804 (patch) | |
tree | fb8d79bf1d0bbfde46384227992d7bfd70942b96 /quantum/action.c | |
parent | 6f2a1e4e17fcba9183c6f4f126383d99d9d714f8 (diff) | |
download | qmk_firmware-8a332e6f0105d2db9239e3c3f997bae754522804.tar.gz qmk_firmware-8a332e6f0105d2db9239e3c3f997bae754522804.zip |
Fix Mod-Tap combo regression (#20669)
* Add keyevent for combo keyrecord
* Fix formatting
* Update quantum/process_keycode/process_combo.c
Co-authored-by: Sergey Vlasov <[email protected]>
* Add combo unit-tests and hot-fix process_record_tap_hint
...as this function tries to lookup the combo keys passed in. This will
be refactored in a later pr.
---------
Co-authored-by: Sergey Vlasov <[email protected]>
Co-authored-by: Stefan Kerkmann <[email protected]>
Diffstat (limited to 'quantum/action.c')
-rw-r--r-- | quantum/action.c | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/quantum/action.c b/quantum/action.c index 9a6bbcca11..41c204f689 100644 --- a/quantum/action.c +++ b/quantum/action.c @@ -187,7 +187,7 @@ bool is_swap_hands_on(void) { */ void process_hand_swap(keyevent_t *event) { keypos_t pos = event->key; - if (pos.row < MATRIX_ROWS && pos.col < MATRIX_COLS) { + if (IS_KEYEVENT(*event) && pos.row < MATRIX_ROWS && pos.col < MATRIX_COLS) { static uint8_t matrix_swap_state[((MATRIX_ROWS * MATRIX_COLS) + (CHAR_BIT)-1) / (CHAR_BIT)]; size_t index = (size_t)(pos.row * MATRIX_COLS) + pos.col; bool do_swap = should_swap_hands(index, matrix_swap_state, event->pressed); @@ -200,7 +200,7 @@ void process_hand_swap(keyevent_t *event) { } } # ifdef ENCODER_MAP_ENABLE - else if (pos.row == KEYLOC_ENCODER_CW || pos.row == KEYLOC_ENCODER_CCW) { + else if (IS_ENCODEREVENT(*event) && pos.row == KEYLOC_ENCODER_CW || pos.row == KEYLOC_ENCODER_CCW) { static uint8_t encoder_swap_state[((NUM_ENCODERS) + (CHAR_BIT)-1) / (CHAR_BIT)]; size_t index = pos.col; bool do_swap = should_swap_hands(index, encoder_swap_state, event->pressed); @@ -242,6 +242,10 @@ __attribute__((weak)) void post_process_record_quantum(keyrecord_t *record) {} * FIXME: Needs documentation. */ void process_record_tap_hint(keyrecord_t *record) { + if (!IS_KEYEVENT(record->event)) { + return; + } + action_t action = layer_switch_get_action(record->event.key); switch (action.kind.id) { |