Remove some assumptions on sequential keycode ranges (#18838)
This commit is contained in:
		
							parent
							
								
									e863c431f6
								
							
						
					
					
						commit
						4f63be4d1d
					
				
					 3 changed files with 25 additions and 6 deletions
				
			
		| 
						 | 
				
			
			@ -316,8 +316,15 @@ led_config_t g_led_config = { {
 | 
			
		|||
#ifdef AUDIO_ENABLE
 | 
			
		||||
bool music_mask_kb(uint16_t keycode) {
 | 
			
		||||
    switch (keycode) {
 | 
			
		||||
        case QK_LAYER_TAP ... QK_ONE_SHOT_LAYER_MAX:
 | 
			
		||||
        case QK_LAYER_TAP_TOGGLE ... QK_LAYER_MOD_MAX:
 | 
			
		||||
        case QK_LAYER_TAP ... QK_LAYER_TAP_MAX:
 | 
			
		||||
        case QK_TO ... QK_TO_MAX:
 | 
			
		||||
        case QK_MOMENTARY ... QK_MOMENTARY_MAX:
 | 
			
		||||
        case QK_DEF_LAYER ... QK_DEF_LAYER_MAX:
 | 
			
		||||
        case QK_TOGGLE_LAYER ... QK_TOGGLE_LAYER_MAX:
 | 
			
		||||
        case QK_ONE_SHOT_LAYER ... QK_ONE_SHOT_LAYER_MAX:
 | 
			
		||||
        case QK_LAYER_TAP_TOGGLE ... QK_LAYER_TAP_TOGGLE_MAX:
 | 
			
		||||
        case QK_LAYER_MOD ... QK_LAYER_MOD_MAX:
 | 
			
		||||
        case QK_ONE_SHOT_MOD ... QK_ONE_SHOT_MOD_MAX:
 | 
			
		||||
        case QK_MOD_TAP ... QK_MOD_TAP_MAX:
 | 
			
		||||
        case AU_ON ... MUV_DE:
 | 
			
		||||
        case QK_BOOT:
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
| 
						 | 
				
			
			@ -318,8 +318,15 @@ bool process_record_kb(uint16_t keycode, keyrecord_t *record) {
 | 
			
		|||
#ifdef AUDIO_ENABLE
 | 
			
		||||
bool music_mask_kb(uint16_t keycode) {
 | 
			
		||||
    switch (keycode) {
 | 
			
		||||
    case QK_LAYER_TAP ... QK_ONE_SHOT_LAYER_MAX:
 | 
			
		||||
    case QK_LAYER_TAP_TOGGLE ... QK_LAYER_MOD_MAX:
 | 
			
		||||
    case QK_LAYER_TAP ... QK_LAYER_TAP_MAX:
 | 
			
		||||
    case QK_TO ... QK_TO_MAX:
 | 
			
		||||
    case QK_MOMENTARY ... QK_MOMENTARY_MAX:
 | 
			
		||||
    case QK_DEF_LAYER ... QK_DEF_LAYER_MAX:
 | 
			
		||||
    case QK_TOGGLE_LAYER ... QK_TOGGLE_LAYER_MAX:
 | 
			
		||||
    case QK_ONE_SHOT_LAYER ... QK_ONE_SHOT_LAYER_MAX:
 | 
			
		||||
    case QK_LAYER_TAP_TOGGLE ... QK_LAYER_TAP_TOGGLE_MAX:
 | 
			
		||||
    case QK_LAYER_MOD ... QK_LAYER_MOD_MAX:
 | 
			
		||||
    case QK_ONE_SHOT_MOD ... QK_ONE_SHOT_MOD_MAX:
 | 
			
		||||
    case QK_MOD_TAP ... QK_MOD_TAP_MAX:
 | 
			
		||||
    case AU_ON ... MUV_DE:
 | 
			
		||||
    case QK_BOOT:
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
| 
						 | 
				
			
			@ -73,8 +73,13 @@ __attribute__((weak)) bool process_autocorrect_user(uint16_t *keycode, keyrecord
 | 
			
		|||
        case KC_LSFT:
 | 
			
		||||
        case KC_RSFT:
 | 
			
		||||
        case KC_CAPS:
 | 
			
		||||
        case QK_TO ... QK_ONE_SHOT_LAYER_MAX:
 | 
			
		||||
        case QK_LAYER_TAP_TOGGLE ... QK_LAYER_MOD_MAX:
 | 
			
		||||
        case QK_TO ... QK_TO_MAX:
 | 
			
		||||
        case QK_MOMENTARY ... QK_MOMENTARY_MAX:
 | 
			
		||||
        case QK_DEF_LAYER ... QK_DEF_LAYER_MAX:
 | 
			
		||||
        case QK_TOGGLE_LAYER ... QK_TOGGLE_LAYER_MAX:
 | 
			
		||||
        case QK_ONE_SHOT_LAYER ... QK_ONE_SHOT_LAYER_MAX:
 | 
			
		||||
        case QK_LAYER_TAP_TOGGLE ... QK_LAYER_TAP_TOGGLE_MAX:
 | 
			
		||||
        case QK_LAYER_MOD ... QK_LAYER_MOD_MAX:
 | 
			
		||||
        case QK_ONE_SHOT_MOD ... QK_ONE_SHOT_MOD_MAX:
 | 
			
		||||
            return false;
 | 
			
		||||
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
		Loading…
	
	Add table
		Add a link
		
	
		Reference in a new issue