1
0
Fork 0

Refactored as well as added support for action keys in combos

This commit is contained in:
Ofer Plesser 2016-12-16 21:50:28 +02:00
parent b6bf4e0dce
commit 6e7cfa83b9
2 changed files with 90 additions and 69 deletions

View file

@ -1,33 +1,33 @@
#include "process_combo.h" #include "process_combo.h"
#include "print.h" #include "print.h"
#define SEND_KEY(key) \
do { \
register_code16(key); \
send_keyboard_report(); \
unregister_code16(key); \
} while(0)
#define COMBO_TIMER_ELAPSED -1 #define COMBO_TIMER_ELAPSED -1
#if COMBO_TERM
#define IS_COMBO_KEY_HELD(combo) (COMBO_TIMER_ELAPSED == combo->timer ? false : true)
#define RESET_COMBO_TIMER_AND_KEY(combo) combo->timer = 0; combo->key = 0
#else
#define IS_COMBO_KEY_HELD(combo) (true)
#define RESET_COMBO_TIMER_AND_KEY(combo) do {} while (0)
#endif
__attribute__ ((weak)) __attribute__ ((weak))
combo_t key_combos[COMBO_COUNT] = { combo_t key_combos[] = {
}; };
static inline void reset_combo(combo_t *combo) __attribute__ ((weak))
void process_combo_event(uint8_t combo_index, bool pressed) {
}
static uint8_t current_combo_index = 0;
static inline void send_combo(uint16_t action, bool pressed)
{ {
combo->state = 0; if (action) {
RESET_COMBO_TIMER_AND_KEY(combo); if (pressed) {
register_code16(action);
} else {
unregister_code16(action);
}
} else {
process_combo_event(current_combo_index, pressed);
}
} }
#define ALL_COMBO_KEYS_ARE_DOWN (((1<<count)-1) == combo->state) #define ALL_COMBO_KEYS_ARE_DOWN (((1<<count)-1) == combo->state)
@ -46,42 +46,51 @@ static bool process_single_combo(combo_t *combo, uint16_t keycode, keyrecord_t *
} }
/* Return if not a combo key */ /* Return if not a combo key */
if (-1 == index) return false; if (-1 == (int8_t)index) return false;
bool is_combo_active = IS_COMBO_KEY_HELD(combo); /* The combos timer is used to signal whether the combo is active */
bool is_combo_active = COMBO_TIMER_ELAPSED == combo->timer ? false : true;
if (record->event.pressed) { if (record->event.pressed) {
KEY_STATE_DOWN(index); KEY_STATE_DOWN(index);
#if COMBO_TERM
if (is_combo_active) { if (is_combo_active) {
if (ALL_COMBO_KEYS_ARE_DOWN) { /* Combo was pressed */
send_combo(combo->keycode, true);
combo->timer = COMBO_TIMER_ELAPSED;
} else { /* Combo key was pressed */
combo->timer = timer_read(); combo->timer = timer_read();
combo->key = keycode; #ifdef COMBO_ALLOW_ACTION_KEYS
} combo->prev_record = *record;
#else
combo->prev_key = keycode;
#endif #endif
}
}
} else { } else {
if (is_combo_active && combo->state) { /* Combo key was tapped */ if (ALL_COMBO_KEYS_ARE_DOWN) { /* Combo was released */
RESET_COMBO_TIMER_AND_KEY(combo); send_combo(combo->keycode, false);
SEND_KEY(keycode);
} }
#if COMBO_TERM if (is_combo_active) { /* Combo key was tapped */
if (!is_combo_active && keycode == combo->key) { /* Held combo key was released */ #ifdef COMBO_ALLOW_ACTION_KEYS
unregister_code16(combo->key); record->event.pressed = true;
} process_action(record, store_or_get_action(record->event.pressed, record->event.key));
record->event.pressed = false;
process_action(record, store_or_get_action(record->event.pressed, record->event.key));
#else
register_code16(keycode);
send_keyboard_report();
unregister_code16(keycode);
#endif #endif
combo->timer = 0;
}
KEY_STATE_UP(index); KEY_STATE_UP(index);
} }
if (ALL_COMBO_KEYS_ARE_DOWN && is_combo_active) { if (NO_COMBO_KEYS_ARE_DOWN) {
SEND_KEY(combo->action); combo->timer = 0;
reset_combo(combo);
}
if(NO_COMBO_KEYS_ARE_DOWN && !is_combo_active) {
reset_combo(combo);
} }
return is_combo_active; return is_combo_active;
@ -91,8 +100,8 @@ bool process_combo(uint16_t keycode, keyrecord_t *record)
{ {
bool is_combo_key = false; bool is_combo_key = false;
for (int i = 0; i < COMBO_COUNT; ++i) { for (current_combo_index = 0; current_combo_index < COMBO_COUNT; ++current_combo_index) {
combo_t *combo = &key_combos[i]; combo_t *combo = &key_combos[current_combo_index];
is_combo_key |= process_single_combo(combo, keycode, record); is_combo_key |= process_single_combo(combo, keycode, record);
} }
@ -101,17 +110,25 @@ bool process_combo(uint16_t keycode, keyrecord_t *record)
void matrix_scan_combo(void) void matrix_scan_combo(void)
{ {
#if COMBO_TERM
for (int i = 0; i < COMBO_COUNT; ++i) { for (int i = 0; i < COMBO_COUNT; ++i) {
combo_t *combo = &key_combos[i]; combo_t *combo = &key_combos[i];
if (combo->timer && if (combo->timer &&
combo->timer != COMBO_TIMER_ELAPSED && combo->timer != COMBO_TIMER_ELAPSED &&
timer_elapsed(combo->timer) > COMBO_TERM) { timer_elapsed(combo->timer) > COMBO_TERM) {
/* This disables the combo, meaning key events for this
* combo will be handled by the next processors in the chain
*/
combo->timer = COMBO_TIMER_ELAPSED; combo->timer = COMBO_TIMER_ELAPSED;
unregister_code16(combo->key);
register_code16(combo->key); #ifdef COMBO_ALLOW_ACTION_KEYS
} process_action(&combo->prev_record,
} store_or_get_action(combo->prev_record.event.pressed,
combo->prev_record.event.key));
#else
unregister_code16(combo->prev_key);
register_code16(combo->prev_key);
#endif #endif
}
}
} }

View file

@ -5,35 +5,39 @@
#include "progmem.h" #include "progmem.h"
#include "quantum.h" #include "quantum.h"
#ifndef COMBO_TERM
#define COMBO_TERM TAPPING_TERM
#endif
typedef struct typedef struct
{ {
const uint16_t *keys; const uint16_t *keys;
uint16_t action; uint16_t keycode;
#ifdef EXTRA_EXTRA_LONG_COMBOS
uint32_t state; uint32_t state;
#if COMBO_TERM #elif EXTRA_LONG_COMBOS
uint16_t state;
#else
uint8_t state;
#endif
uint16_t timer; uint16_t timer;
uint16_t key; #ifdef COMBO_ALLOW_ACTION_KEYS
keyrecord_t prev_record;
#else
uint16_t prev_key;
#endif #endif
} combo_t; } combo_t;
#if COMBO_TERM #define COMBO(ck, ca) {.keys = &(ck)[0], .keycode = (ca)}
#define COMBO(ck, ca) {.keys = &(ck)[0], .action = (ca), .state = 0, .timer = 0, .key = 0} #define COMBO_ACTION(ck) {.keys = &(ck)[0]}
#else
#define COMBO(ck, ca) {.keys = &(ck)[0], .action = (ca), .state = 0 }
#endif
#define COMBO_END 0 #define COMBO_END 0
#ifndef COMBO_COUNT #ifndef COMBO_COUNT
#define COMBO_COUNT 0 #define COMBO_COUNT 0
#endif #endif
#ifndef COMBO_TERM
extern combo_t key_combos[COMBO_COUNT]; #define COMBO_TERM TAPPING_TERM
#endif
bool process_combo(uint16_t keycode, keyrecord_t *record); bool process_combo(uint16_t keycode, keyrecord_t *record);
void matrix_scan_combo(void); void matrix_scan_combo(void);
void process_combo_event(uint8_t combo_index, bool pressed);
#endif #endif