diff options
Diffstat (limited to 'quantum')
| -rw-r--r-- | quantum/action.c | 18 | ||||
| -rw-r--r-- | quantum/action_tapping.c | 4 | ||||
| -rw-r--r-- | quantum/keycode.h | 2 | ||||
| -rw-r--r-- | quantum/keycodes.h | 2 | ||||
| -rw-r--r-- | quantum/process_keycode/process_combo.h | 2 | ||||
| -rw-r--r-- | quantum/process_keycode/process_key_override.c | 2 | ||||
| -rw-r--r-- | quantum/process_keycode/process_space_cadet.c | 10 | ||||
| -rw-r--r-- | quantum/quantum.c | 4 | 
8 files changed, 21 insertions, 23 deletions
| diff --git a/quantum/action.c b/quantum/action.c index 6b5660af8b..72f132eaa8 100644 --- a/quantum/action.c +++ b/quantum/action.c @@ -351,7 +351,7 @@ void process_action(keyrecord_t *record, action_t action) {      bool do_release_oneshot = false;      // notice we only clear the one shot layer if the pressed key is not a modifier.      if (is_oneshot_layer_active() && event.pressed && -        (action.kind.id == ACT_USAGE || !(IS_MOD(action.key.code) +        (action.kind.id == ACT_USAGE || !(IS_MODIFIER_KEYCODE(action.key.code)  #    ifndef NO_ACTION_TAPPING                                            || (tap_count == 0 && (action.kind.id == ACT_LMODS_TAP || action.kind.id == ACT_RMODS_TAP))  #    endif @@ -372,7 +372,7 @@ void process_action(keyrecord_t *record, action_t action) {              uint8_t mods = (action.kind.id == ACT_LMODS) ? action.key.mods : action.key.mods << 4;              if (event.pressed) {                  if (mods) { -                    if (IS_MOD(action.key.code) || action.key.code == KC_NO) { +                    if (IS_MODIFIER_KEYCODE(action.key.code) || action.key.code == KC_NO) {                          // e.g. LSFT(KC_LEFT_GUI): we don't want the LSFT to be weak as it would make it useless.                          // This also makes LSFT(KC_LEFT_GUI) behave exactly the same as LGUI(KC_LEFT_SHIFT).                          // Same applies for some keys like KC_MEH which are declared as MEH(KC_NO). @@ -386,7 +386,7 @@ void process_action(keyrecord_t *record, action_t action) {              } else {                  unregister_code(action.key.code);                  if (mods) { -                    if (IS_MOD(action.key.code) || action.key.code == KC_NO) { +                    if (IS_MODIFIER_KEYCODE(action.key.code) || action.key.code == KC_NO) {                          del_mods(mods);                      } else {                          del_weak_mods(mods); @@ -406,7 +406,7 @@ void process_action(keyrecord_t *record, action_t action) {                      if (!keymap_config.oneshot_enable) {                          if (event.pressed) {                              if (mods) { -                                if (IS_MOD(action.key.code) || action.key.code == KC_NO) { +                                if (IS_MODIFIER_KEYCODE(action.key.code) || action.key.code == KC_NO) {                                      // e.g. LSFT(KC_LGUI): we don't want the LSFT to be weak as it would make it useless.                                      // This also makes LSFT(KC_LGUI) behave exactly the same as LGUI(KC_LSFT).                                      // Same applies for some keys like KC_MEH which are declared as MEH(KC_NO). @@ -420,7 +420,7 @@ void process_action(keyrecord_t *record, action_t action) {                          } else {                              unregister_code(action.key.code);                              if (mods) { -                                if (IS_MOD(action.key.code) || action.key.code == KC_NO) { +                                if (IS_MODIFIER_KEYCODE(action.key.code) || action.key.code == KC_NO) {                                      del_mods(mods);                                  } else {                                      del_weak_mods(mods); @@ -877,7 +877,7 @@ __attribute__((weak)) void register_code(uint8_t code) {          send_keyboard_report();  #endif -    } else if IS_KEY (code) { +    } else if IS_BASIC_KEYCODE (code) {          // TODO: should push command_proc out of this block?          if (command_proc(code)) return; @@ -890,7 +890,7 @@ __attribute__((weak)) void register_code(uint8_t code) {          }          add_key(code);          send_keyboard_report(); -    } else if IS_MOD (code) { +    } else if IS_MODIFIER_KEYCODE (code) {          add_mods(MOD_BIT(code));          send_keyboard_report(); @@ -944,10 +944,10 @@ __attribute__((weak)) void unregister_code(uint8_t code) {          send_keyboard_report();  #endif -    } else if IS_KEY (code) { +    } else if IS_BASIC_KEYCODE (code) {          del_key(code);          send_keyboard_report(); -    } else if IS_MOD (code) { +    } else if IS_MODIFIER_KEYCODE (code) {          del_mods(MOD_BIT(code));          send_keyboard_report(); diff --git a/quantum/action_tapping.c b/quantum/action_tapping.c index 821265d399..60c7ed4973 100644 --- a/quantum/action_tapping.c +++ b/quantum/action_tapping.c @@ -247,12 +247,12 @@ bool process_tapping(keyrecord_t *keyp) {                          case ACT_LMODS:                          case ACT_RMODS:                              if (action.key.mods && !action.key.code) return false; -                            if (IS_MOD(action.key.code)) return false; +                            if (IS_MODIFIER_KEYCODE(action.key.code)) return false;                              break;                          case ACT_LMODS_TAP:                          case ACT_RMODS_TAP:                              if (action.key.mods && keyp->tap.count == 0) return false; -                            if (IS_MOD(action.key.code)) return false; +                            if (IS_MODIFIER_KEYCODE(action.key.code)) return false;                              break;                          case ACT_LAYER_TAP:                          case ACT_LAYER_TAP_EXT: diff --git a/quantum/keycode.h b/quantum/keycode.h index 45736e92f1..03989ed8f6 100644 --- a/quantum/keycode.h +++ b/quantum/keycode.h @@ -27,8 +27,6 @@ along with this program.  If not, see <http://www.gnu.org/licenses/>.  /* FIXME: Add doxygen comments here */  #define IS_ANY(code) (KC_A <= (code) && (code) <= 0xFF) -#define IS_KEY(code) IS_BASIC_KEYCODE(code) -#define IS_MOD(code) IS_MODIFIERS_KEYCODE(code)  #define IS_SYSTEM(code) IS_SYSTEM_KEYCODE(code)  #define IS_CONSUMER(code) IS_MEDIA_KEYCODE(code) diff --git a/quantum/keycodes.h b/quantum/keycodes.h index 96f352a2c1..f24ccf01b8 100644 --- a/quantum/keycodes.h +++ b/quantum/keycodes.h @@ -1321,7 +1321,7 @@ enum qk_keycode_defines {  #define IS_SYSTEM_KEYCODE(code) ((code) >= KC_SYSTEM_POWER && (code) <= KC_SYSTEM_WAKE)  #define IS_MEDIA_KEYCODE(code) ((code) >= KC_AUDIO_MUTE && (code) <= KC_ASSISTANT)  #define IS_MOUSE_KEYCODE(code) ((code) >= KC_MS_UP && (code) <= KC_MS_ACCEL2) -#define IS_MODIFIERS_KEYCODE(code) ((code) >= KC_LEFT_CTRL && (code) <= KC_RIGHT_GUI) +#define IS_MODIFIER_KEYCODE(code) ((code) >= KC_LEFT_CTRL && (code) <= KC_RIGHT_GUI)  #define IS_SWAP_HANDS_KEYCODE(code) ((code) >= QK_SWAP_HANDS_TOGGLE && (code) <= QK_SWAP_HANDS_ONE_SHOT)  #define IS_MAGIC_KEYCODE(code) ((code) >= MAGIC_SWAP_CONTROL_CAPSLOCK && (code) <= MAGIC_TOGGLE_ESCAPE_CAPSLOCK)  #define IS_MIDI_KEYCODE(code) ((code) >= QK_MIDI_ON && (code) <= QK_MIDI_PITCH_BEND_UP) diff --git a/quantum/process_keycode/process_combo.h b/quantum/process_keycode/process_combo.h index 4c4e574e34..e430c4a5f7 100644 --- a/quantum/process_keycode/process_combo.h +++ b/quantum/process_keycode/process_combo.h @@ -69,7 +69,7 @@ typedef struct {  #endif  /* check if keycode is only modifiers */ -#define KEYCODE_IS_MOD(code) (IS_MOD(code) || (code >= QK_MODS && code <= QK_MODS_MAX && !(code & QK_BASIC_MAX))) +#define KEYCODE_IS_MOD(code) (IS_MODIFIER_KEYCODE(code) || (IS_QK_MODS(code) && !QK_MODS_GET_BASIC_KEYCODE(code)))  bool process_combo(uint16_t keycode, keyrecord_t *record);  void combo_task(void); diff --git a/quantum/process_keycode/process_key_override.c b/quantum/process_keycode/process_key_override.c index 9c5abccd4f..9b91ae2223 100644 --- a/quantum/process_keycode/process_key_override.c +++ b/quantum/process_keycode/process_key_override.c @@ -402,7 +402,7 @@ bool process_key_override(const uint16_t keycode, const keyrecord_t *const recor  #endif      const bool key_down = record->event.pressed; -    const bool is_mod   = IS_MOD(keycode); +    const bool is_mod   = IS_MODIFIER_KEYCODE(keycode);      if (key_down) {          switch (keycode) { diff --git a/quantum/process_keycode/process_space_cadet.c b/quantum/process_keycode/process_space_cadet.c index a62cd60a70..3109ea1711 100644 --- a/quantum/process_keycode/process_space_cadet.c +++ b/quantum/process_keycode/process_space_cadet.c @@ -89,16 +89,16 @@ void perform_space_cadet(keyrecord_t *record, uint16_t sc_keycode, uint8_t holdM  #ifdef SPACE_CADET_MODIFIER_CARRYOVER          sc_mods = get_mods();  #endif -        if (IS_MOD(holdMod)) { +        if (IS_MODIFIER_KEYCODE(holdMod)) {              register_mods(MOD_BIT(holdMod));          }      } else {          if (sc_last == holdMod && timer_elapsed(sc_timer) < GET_TAPPING_TERM(sc_keycode, record)) {              if (holdMod != tapMod) { -                if (IS_MOD(holdMod)) { +                if (IS_MODIFIER_KEYCODE(holdMod)) {                      unregister_mods(MOD_BIT(holdMod));                  } -                if (IS_MOD(tapMod)) { +                if (IS_MODIFIER_KEYCODE(tapMod)) {                      register_mods(MOD_BIT(tapMod));                  }              } @@ -109,11 +109,11 @@ void perform_space_cadet(keyrecord_t *record, uint16_t sc_keycode, uint8_t holdM  #ifdef SPACE_CADET_MODIFIER_CARRYOVER              clear_weak_mods();  #endif -            if (IS_MOD(tapMod)) { +            if (IS_MODIFIER_KEYCODE(tapMod)) {                  unregister_mods(MOD_BIT(tapMod));              }          } else { -            if (IS_MOD(holdMod)) { +            if (IS_MODIFIER_KEYCODE(holdMod)) {                  unregister_mods(MOD_BIT(holdMod));              }          } diff --git a/quantum/quantum.c b/quantum/quantum.c index 0e69246f41..653f553f60 100644 --- a/quantum/quantum.c +++ b/quantum/quantum.c @@ -76,7 +76,7 @@ void do_code16(uint16_t code, void (*f)(uint8_t)) {  }  __attribute__((weak)) void register_code16(uint16_t code) { -    if (IS_MOD(code) || code == KC_NO) { +    if (IS_MODIFIER_KEYCODE(code) || code == KC_NO) {          do_code16(code, register_mods);      } else {          do_code16(code, register_weak_mods); @@ -86,7 +86,7 @@ __attribute__((weak)) void register_code16(uint16_t code) {  __attribute__((weak)) void unregister_code16(uint16_t code) {      unregister_code(code); -    if (IS_MOD(code) || code == KC_NO) { +    if (IS_MODIFIER_KEYCODE(code) || code == KC_NO) {          do_code16(code, unregister_mods);      } else {          do_code16(code, unregister_weak_mods); | 
