Ergodox EZ refactor (#3171)

* Refactor matrices and keymaps

* Fixes for @ericgebhart keymap

* Deleted macro aliases

* ericgebhart/keymap.c to LAYOUT_ergodox_wrapper

1 files changed, 99 insertions, 99 deletions

diff --git a/keyboards/ergodox_ez/keymaps/dvorak_42_key/keymap.c b/keyboards/ergodox_ez/keymaps/dvorak_42_key/keymap.c
index e9959b9966..088449e2fe 100644
--- a/keyboards/ergodox_ez/keymaps/dvorak_42_key/keymap.c
+++ b/keyboards/ergodox_ez/keymaps/dvorak_42_key/keymap.c
@@ -25,13 +25,13 @@ enum custom_keycodes {
   // shell nav macros
   SHELL_LS,
   SHELL_LSLTR,
-  SHELL_LSLA, 
+  SHELL_LSLA,
   SHELL_CDPRE,
   SHELL_LESS,
   SHELL_PLESS,
   SHELL_PGREP,
-  SHELL_TAILF,  
-  
+  SHELL_TAILF,
+
   SHELL_PWD,
   SHELL_H3,
   SHELL_AMMCOLO,
@@ -40,7 +40,7 @@ enum custom_keycodes {
   SHELL_SCREEN_LIST,
   SHELL_MKE,
   SHELL_HTCSTATUS,
-  
+
   // Cloud9 macros
   CLOUD9_TAB_LEFT,
   CLOUD9_TAB_RIGHT,
@@ -48,7 +48,7 @@ enum custom_keycodes {
   CLOUD9_GOTO_SYMBOL,
   CLOUD9_GOTO_LINE,
   CLOUD9_NAVIGATE,
-  
+
 };
 
 
@@ -57,7 +57,7 @@ enum custom_keycodes {
 #define KEYSEL       2 // arrow navigation + shift (allow text selection)
 #define SHELL_NAV    3 // bash shortcuts
 #define SHELL_SCREEN 4 // linux screen shortcuts
-#define SCREEN_NAV   5 // navigate between linux screen tabs 
+#define SCREEN_NAV   5 // navigate between linux screen tabs
 #define MOUSE        6 // mouse layer (can be locked with lock key), unmapped for now
 #define BROWSER_CONTROL 7 // control browser and mouse
 #define COMBINED      	8 // combined numbers and symbols layer
@@ -105,7 +105,7 @@ enum custom_keycodes {
                 return MACRO( D(LCTL), T(A), U(LCTL), T(NUM), END);\
             }\
         break;\
-		
+
 #define MACRO_SCREEN_REG(MACRO_NAME,NUM) \
         case MACRO_NAME:\
              if (record->event.pressed) {\
@@ -142,32 +142,32 @@ enum custom_keycodes {
 
 const uint16_t PROGMEM keymaps[][MATRIX_ROWS][MATRIX_COLS] = {
 
-  [BASE] = KEYMAP(
+  [BASE] = LAYOUT_ergodox(
       // left hand
       KC_ESC,            KC_F1,         KC_F2,      KC_F3,        KC_F4,   KC_F5,   KC_F6,
       OSL(SCREEN_NAV),   KC_QUOTE,      KC_COMMA,   KC_DOT,       KC_P,    KC_Y,    MEH(KC_2),
       OSL(SHELL_NAV),    KC_A,          KC_O,       KC_E,         KC_U,    KC_I,
       OSL(SHELL_SCREEN), KC_SCOLON,     KC_Q,       KC_J,         KC_K,    KC_X,    MEH(KC_3),
       MEH(KC_1),         OSM(MOD_LSFT), OSM(MOD_LCTL), MO(KEYSEL), MO(BROWSER_CONTROL),
-      
+
       // left thumb cluster
                 MEH(KC_4),      MEH(KC_5),
                                 MEH(KC_6),
       MO(COMBINED),MO(KEYNAV),  OSM(MOD_LALT),
-      
+
       // right hand
       KC_F7,     KC_F8, KC_F9, KC_F10, KC_F11, KC_F12, MEH(KC_9),
       MEH(KC_7), KC_F,  KC_G,  KC_C,   KC_R,   KC_L,   KC_TAB,
                  KC_D,  KC_H,  KC_T,   KC_N,   KC_S,   RCTL(KC_BSPC),
       MEH(KC_8), KC_B,  KC_M,  KC_W,   KC_V,   KC_Z,   MEH(KC_F3),
       KC_BSPC,   RCTL(KC_BSPC), KC_CAPSLOCK, OSM(MOD_LSFT),MEH(KC_F4),
-    
+
       // right thumb cluster
       MEH(KC_F5),MEH(KC_F6),MEH(KC_F7),MEH(KC_F8),KC_ENTER,KC_SPACE
-  
+
   ),
-  
-  [KEYNAV] = KEYMAP(
+
+  [KEYNAV] = LAYOUT_ergodox(
     // left hand
     KC_TRNS,KC_TRNS,     KC_TRNS,            KC_TRNS,         KC_TRNS,          KC_TRNS,          KC_TRNS,
     KC_TRNS,KC_ESC,      CLOUD9_GOTO_LINE,   RCTL(KC_Z),      RCTL(KC_S),       MEH(KC_F10),      KC_TRNS,
@@ -176,20 +176,20 @@ const uint16_t PROGMEM keymaps[][MATRIX_ROWS][MATRIX_COLS] = {
     KC_TRNS,KC_TRNS,     KC_TRNS,            KC_TRNS,         KC_TRNS,
     // left thumb cluster
     KC_TRNS,KC_TRNS,KC_TRNS,TO(MOUSE),KC_TRNS,KC_TRNS,
-    
+
     // right hand
     KC_TRNS,KC_TRNS,       KC_TRNS,    KC_TRNS,         KC_TRNS,    KC_TRNS,         MEH(KC_G),
     KC_TRNS,KC_NO,         KC_HOME,    KC_UP,           KC_END,     KC_PGUP,         MEH(KC_H),
             LCTL(KC_LEFT), KC_LEFT,    KC_DOWN,         KC_RIGHT,   LCTL(KC_RIGHT),  MEH(KC_I),
     KC_TRNS,KC_NO,         RCTL(KC_C), RCTL(KC_X),      RCTL(KC_V), KC_PGDOWN,       MEH(KC_J),
                            KC_BSPC,   RCTL(KC_BSPC),    KC_DELETE,  LCTL(KC_DELETE), MEH(KC_K),
-    
+
     // right thumb cluster
     KC_TRNS,KC_TRNS,KC_TRNS,KC_TRNS,KC_TRNS,KC_TRNS
-  ),  
-  
+  ),
+
     // key selection layer
-    [KEYSEL] = KEYMAP(
+    [KEYSEL] = LAYOUT_ergodox(
            // left hand
            KC_TRNS,KC_TRNS,   KC_TRNS,  KC_TRNS,   KC_TRNS,   KC_TRNS,   KC_TRNS,
            KC_TRNS,MEH(KC_G), MEH(KC_H),MEH(KC_I), MEH(KC_J), MEH(KC_K), KC_TRNS,
@@ -212,10 +212,10 @@ const uint16_t PROGMEM keymaps[][MATRIX_ROWS][MATRIX_COLS] = {
            KC_TRNS, KC_TRNS,
            KC_TRNS,
            KC_TRNS, KC_TRNS, KC_TRNS
-    ),  
-  
+    ),
+
   // shell navigation layer
-  [SHELL_NAV] = KEYMAP(
+  [SHELL_NAV] = LAYOUT_ergodox(
        // left hand
        KC_NO,  KC_TRNS,           KC_TRNS,        KC_TRNS,          KC_TRNS,     KC_TRNS,         KC_TRNS,
        KC_TRNS,KC_TRNS,           SHELL_PGREP,    SHELL_PLESS,      SHELL_LESS,  KC_TRNS,         SHELL_H3,
@@ -239,10 +239,10 @@ const uint16_t PROGMEM keymaps[][MATRIX_ROWS][MATRIX_COLS] = {
        KC_TRNS,
        KC_TRNS, KC_TRNS, KC_TRNS
   ),
-  
+
 
     // linux screen layer
-  [SHELL_SCREEN] = KEYMAP(
+  [SHELL_SCREEN] = LAYOUT_ergodox(
        // left hand
        KC_NO,  KC_TRNS,    KC_TRNS,     KC_TRNS,     KC_TRNS,     KC_TRNS,   KC_TRNS,
        KC_TRNS,KC_TRNS,    KC_TRNS,     KC_TRNS,     KC_TRNS,     KC_TRNS,   KC_TRNS,
@@ -268,7 +268,7 @@ const uint16_t PROGMEM keymaps[][MATRIX_ROWS][MATRIX_COLS] = {
   ),
 
   // navigation within screen (for copy/paste)
-  [SCREEN_NAV] = KEYMAP(
+  [SCREEN_NAV] = LAYOUT_ergodox(
        // left hand
        // left hand
        KC_NO,  KC_TRNS,    KC_TRNS,              KC_TRNS,     		  KC_TRNS,               KC_TRNS,   KC_TRNS,
@@ -293,12 +293,12 @@ const uint16_t PROGMEM keymaps[][MATRIX_ROWS][MATRIX_COLS] = {
        KC_TRNS,
        KC_TRNS, KC_TRNS, KC_TRNS
   ),
-  
 
 
-  
-  [COMBINED] = KEYMAP(
-  
+
+
+  [COMBINED] = LAYOUT_ergodox(
+
     // left hand
     KC_NO,  KC_TRNS, KC_TRNS,    KC_TRNS,     KC_TRNS,     KC_TRNS, KC_TRNS,
     KC_TRNS,KC_EXLM, KC_AT,      KC_HASH,     KC_DLR,      KC_PERC, KC_TRNS,
@@ -308,27 +308,27 @@ const uint16_t PROGMEM keymaps[][MATRIX_ROWS][MATRIX_COLS] = {
     KC_TRNS,KC_TRNS, KC_TILD,    KC_GRAVE,    KC_CIRC,
     // thumb cluster
     KC_TRNS,KC_TRNS,KC_TRNS,KC_TRNS,KC_TRNS,KC_TRNS,
-    
+
     // right hand
     KC_TRNS, KC_TRNS,     KC_TRNS, KC_TRNS, KC_TRNS, KC_TRNS,     MEH(KC_L),
     KC_TRNS, KC_PLUS,     KC_7,    KC_8,    KC_9,    KC_ASTR,     MEH(KC_M),
              KC_MINS,     KC_4,    KC_5,    KC_6,    KC_SLSH,     MEH(KC_N),
     KC_TRNS, KC_EQUAL,    KC_1,    KC_2,    KC_3,    KC_QUES,     MEH(KC_O),
-    // bottom row 
+    // bottom row
                           KC_0,    KC_DOT,  KC_PIPE, KC_BSLS,     MEH(KC_P),
     // thumb cluster
     KC_TRNS,KC_TRNS,
 	KC_TRNS,
 	KC_TRNS,KC_TRNS,KC_TRNS),
-  
-  
-    [MOUSE] = KEYMAP(
+
+
+    [MOUSE] = LAYOUT_ergodox(
            KC_TRNS, KC_TRNS,      KC_TRNS,       KC_TRNS,       KC_TRNS,      KC_TRNS, KC_TRNS,
            KC_TRNS, KC_TRNS,      KC_PGUP,       KC_MS_WH_UP,   KC_UP,        KC_TRNS, KC_TRNS,
            KC_TRNS, KC_MS_ACCEL0, KC_PGDN,       KC_MS_WH_DOWN, KC_DOWN,      KC_TRNS,
            KC_TRNS, KC_TRNS,      KC_TRNS,       KC_TRNS,       KC_TRNS,      KC_TRNS, KC_TRNS,
            KC_TRNS, KC_TRNS,      M(MOUSE_LOCK), KC_TRNS,       KC_MS_ACCEL0,
-           
+
                                                KC_TRNS, KC_TRNS,
                                                         KC_TRNS,
                                       KC_TRNS, KC_BTN3, KC_TRNS,
@@ -342,9 +342,9 @@ const uint16_t PROGMEM keymaps[][MATRIX_ROWS][MATRIX_COLS] = {
            KC_TRNS, KC_TRNS,
            KC_TRNS,
            KC_TRNS, KC_BTN1, KC_BTN2
-    ),  
-	
-    [BROWSER_CONTROL] = KEYMAP(
+    ),
+
+    [BROWSER_CONTROL] = LAYOUT_ergodox(
 		   // left hand
            KC_TRNS, KC_TRNS,      KC_TRNS,       KC_TRNS,       KC_TRNS,      KC_TRNS, KC_TRNS,
            KC_TRNS, KC_TRNS,      KC_BTN3,       KC_MS_U,       KC_BTN1,      KC_BTN2, KC_TRNS,
@@ -352,7 +352,7 @@ const uint16_t PROGMEM keymaps[][MATRIX_ROWS][MATRIX_COLS] = {
            KC_TRNS, KC_TRNS,      KC_TRNS,       KC_TRNS,       KC_TRNS,      KC_TRNS, KC_TRNS,
 		   // bottom row
            KC_TRNS, KC_TRNS,      KC_TRNS,       KC_TRNS,       KC_TRNS,
-           
+
                                                KC_TRNS, KC_TRNS,
                                                         KC_TRNS,
                                       KC_TRNS, KC_TRNS, KC_TRNS,
@@ -366,8 +366,8 @@ const uint16_t PROGMEM keymaps[][MATRIX_ROWS][MATRIX_COLS] = {
            KC_TRNS, KC_TRNS,
            KC_TRNS,
            KC_TRNS, RCTL(KC_W), RCTL(KC_T)
-    ),  	
-    
+    ),
+
 
 };
 
@@ -402,28 +402,28 @@ const macro_t *action_get_macro(keyrecord_t *record, uint8_t id, uint8_t opt)
         case SCREEN_TAB_LEFT:
             if (record->event.pressed) {
                 return MACRO( D(LCTL), T(A), U(LCTL), T(P), END);
-            }        
+            }
             break;
         case SCREEN_TAB_RIGHT:
              if (record->event.pressed) {
                 return MACRO( D(LCTL), T(A), U(LCTL), T(N), END);
-            }                    
+            }
             break;
         case SCREEN_NEW_TAB:
              if (record->event.pressed) {
                 return MACRO( D(LCTL), T(A), U(LCTL), T(C), END);
-            }                                
+            }
         break;
         case SCREEN_DETACH:
              if (record->event.pressed) {
                 return MACRO( D(LCTL), T(A), U(LCTL), T(D), END);
             }
-        break;        
+        break;
         case SCREEN_RENAME:
              if (record->event.pressed) {
                 return MACRO( D(LCTL), T(A), U(LCTL), D(LSFT), T(A), U(LSFT), END);
-            }                                
-        break;        
+            }
+        break;
         case SCREEN_NUMBER:
              if (record->event.pressed) {
                 return MACRO( D(LCTL), T(A), U(LCTL), D(LSFT), T(SCOLON), U(LSFT),
@@ -435,13 +435,13 @@ const macro_t *action_get_macro(keyrecord_t *record, uint8_t id, uint8_t opt)
                               T(R),
                               T(SPC),
                              END);
-            }                                
-        break;        
+            }
+        break;
 
 		MACRO_SCREEN_REG(SCREEN_READREG_1,1);
 		MACRO_SCREEN_REG(SCREEN_READREG_2,2);
 		MACRO_SCREEN_REG(SCREEN_READREG_3,3);
-		
+
 		MACRO_SCREEN_PASTE(SCREEN_PASTEREG_1,1);
 		MACRO_SCREEN_PASTE(SCREEN_PASTEREG_2,2);
 		MACRO_SCREEN_PASTE(SCREEN_PASTEREG_3,3);
@@ -456,19 +456,19 @@ const macro_t *action_get_macro(keyrecord_t *record, uint8_t id, uint8_t opt)
         MACRO_SCREEN_NUM(SCREEN_7,7);
         MACRO_SCREEN_NUM(SCREEN_8,8);
         MACRO_SCREEN_NUM(SCREEN_9,9);
-        
+
 		case SCREEN_UP_JUMP:
             if (record->event.pressed) {
                 return MACRO( T(5), T(UP), END);
-            }		
+            }
 		break;
-		
+
 		case SCREEN_DOWN_JUMP:
             if (record->event.pressed) {
                 return MACRO( T(5), T(DOWN), END);
-            }		
-		break;		
-		
+            }
+		break;
+
         case SCREEN_COPY_MODE:
             if (record->event.pressed) {
                 return MACRO( D(LCTL), T(A), U(LCTL), T(ESC), END);
@@ -476,42 +476,42 @@ const macro_t *action_get_macro(keyrecord_t *record, uint8_t id, uint8_t opt)
         break;
         case SCREEN_PASTE:
             if (record->event.pressed) {
-                return MACRO( D(LCTL), T(A), U(LCTL), T(RBRC), END); 
+                return MACRO( D(LCTL), T(A), U(LCTL), T(RBRC), END);
             }
-        break;        
-		
+        break;
+
 		case SHELL_WRITE_LOGPATTERN:
             if (record->event.pressed) {
                 return MACRO( D(LSFT),
-							  T(8), 
-				              T(8), 
+							  T(8),
+				              T(8),
 							  U(LSFT),
-							  T(L),  
-							  T(O), 
-							  T(G), 
-							  T(LEFT), 
-							  T(LEFT), 
-							  T(LEFT), 
-							  T(LEFT), 
-							  END); 
-            }		
+							  T(L),
+							  T(O),
+							  T(G),
+							  T(LEFT),
+							  T(LEFT),
+							  T(LEFT),
+							  T(LEFT),
+							  END);
+            }
 		break;
-		
+
 		case SHELL_WRITE_TRANPATTERN:
             if (record->event.pressed) {
                 return MACRO( D(LSFT),
-							  T(8), 
-				              T(8), 
-							  U(LSFT), T(T), T(R), T(A), T(N), T(LEFT), T(LEFT), T(LEFT), T(LEFT), T(LEFT), END); 
-            }		
-		break;		
-		
+							  T(8),
+				              T(8),
+							  U(LSFT), T(T), T(R), T(A), T(N), T(LEFT), T(LEFT), T(LEFT), T(LEFT), T(LEFT), END);
+            }
+		break;
+
 		case SHELL_EXPAND_PATTERN:
             if (record->event.pressed) {
-                return MACRO( D(LALT), T(F), U(LALT), D(LCTL), T(X), U(LCTL), D(LSFT), T(8), D(LSFT), END); 
-            }		
-		break;				
-   
+                return MACRO( D(LALT), T(F), U(LALT), D(LCTL), T(X), U(LCTL), D(LSFT), T(8), D(LSFT), END);
+            }
+		break;
+
       }
     return MACRO_NONE;
 };
@@ -538,7 +538,7 @@ bool process_record_user(uint16_t keycode, keyrecord_t *record) {
       return false;
       break;
   }
-  
+
   // shell macros
   if(record->event.pressed) {
     switch (keycode) {
@@ -569,7 +569,7 @@ bool process_record_user(uint16_t keycode, keyrecord_t *record) {
         case SHELL_PGREP:
             SEND_STRING(" | grep ");
             return true;
-            break;            
+            break;
         case SHELL_TAILF:
             SEND_STRING("tail -f ");
             return true;
@@ -577,19 +577,19 @@ bool process_record_user(uint16_t keycode, keyrecord_t *record) {
         case SHELL_PWD:
             SEND_STRING("echo `pwd`/");
             return true;
-            break;            
+            break;
         case SHELL_H3:
             SEND_STRING("h3\n");
             return true;
-            break;            
+            break;
         case SHELL_AMMCOLO:
             SEND_STRING("ammcolo\n");
             return true;
-            break;            
+            break;
         case SHELL_SCREENRD:
             SEND_STRING("screen -r -d ");
             return true;
-            break;                    
+            break;
         case SHELL_SCREEN_NEW:
             SEND_STRING("screen -S ");
             return true;
@@ -597,44 +597,44 @@ bool process_record_user(uint16_t keycode, keyrecord_t *record) {
         case SHELL_SCREEN_LIST:
             SEND_STRING("screen -list\n");
             return true;
-            break;            
+            break;
         case SHELL_MKE:
             SEND_STRING("mke\n");
             return true;
-            break;     
+            break;
         case SHELL_HTCSTATUS:
             SEND_STRING("htcStatus -j ");
             return true;
-            break;               
+            break;
 		// Cloud9 macros
 		case CLOUD9_TAB_LEFT:
             SEND_STRING(SS_LCTRL("["));
-            return true;		
+            return true;
 			break;
 		case CLOUD9_TAB_RIGHT:
             SEND_STRING(SS_LCTRL("]"));
-            return true;				
+            return true;
 			break;
 		case CLOUD9_TAB_CLOSE:
             SEND_STRING(SS_LALT("w"));
-            return true;				
+            return true;
 			break;
 		case CLOUD9_GOTO_SYMBOL:
             SEND_STRING(SS_LSFT(SS_LCTRL("e")));
-            return true;		
+            return true;
 			break;
 		case CLOUD9_GOTO_LINE:
             SEND_STRING(SS_LCTRL("g"));
-            return true;						
+            return true;
 			break;
 		case CLOUD9_NAVIGATE:
             SEND_STRING(SS_LCTRL("e"));
             return true;
 			break;
-  		
+
     }
   }
-  
+
   return true;
 }
 
@@ -655,9 +655,9 @@ void matrix_scan_user(void) {
     ergodox_right_led_3_off();
     switch (layer) {
         case COMBINED:
-            ergodox_right_led_2_on();        
+            ergodox_right_led_2_on();
             break;
-        case SHELL_NAV:            
+        case SHELL_NAV:
         case SHELL_SCREEN:
         case KEYNAV:
         case KEYSEL: