From 40397a70bc3d92cdb6e8429745ed1e9a8ec926f8 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?L=C3=A1szl=C3=B3=20Monda?= Date: Sat, 25 Feb 2017 22:13:48 +0100 Subject: [PATCH] Remove check because keyId < MAX_KEY_COUNT_PER_MODULE should always be true. --- right/src/action.c | 18 +++++++----------- 1 file changed, 7 insertions(+), 11 deletions(-) diff --git a/right/src/action.c b/right/src/action.c index 2b33250..e31d429 100644 --- a/right/src/action.c +++ b/right/src/action.c @@ -11,19 +11,15 @@ static uint8_t ActiveLayer = LAYER_ID_BASE; static key_action_t getKeycode(uint8_t slotId, uint8_t keyId) { - if (keyId < MAX_KEY_COUNT_PER_MODULE) { - if (keyMasks[slotId][keyId]!=0 && keyMasks[slotId][keyId]!=ActiveLayer) { - // Mask out key presses after releasing modifier keys - return (key_action_t){.type = KEY_ACTION_NONE}; - } - - key_action_t key = CurrentKeymap[ActiveLayer][slotId][keyId]; - keyMasks[slotId][keyId] = ActiveLayer; - - return key; - } else { + if (keyMasks[slotId][keyId]!=0 && keyMasks[slotId][keyId]!=ActiveLayer) { + // Mask out key presses after releasing modifier keys return (key_action_t){.type = KEY_ACTION_NONE}; } + + key_action_t key = CurrentKeymap[ActiveLayer][slotId][keyId]; + keyMasks[slotId][keyId] = ActiveLayer; + + return key; } static void clearKeymask(const uint8_t *keyStates)