Convert Encoder callbacks to be boolean functions (#12805)

Co-authored-by: James Young <18669334+noroadsleft@users.noreply.github.com>

2 files changed, 20 insertions, 18 deletions

diff --git a/keyboards/pabile/p18/keymaps/default/keymap.c b/keyboards/pabile/p18/keymaps/default/keymap.c
index d47982e562..0223a450cd 100644
--- a/keyboards/pabile/p18/keymaps/default/keymap.c
+++ b/keyboards/pabile/p18/keymaps/default/keymap.c
@@ -2,26 +2,27 @@
 
 const uint16_t PROGMEM keymaps[][MATRIX_ROWS][MATRIX_COLS] = {
     [0] = LAYOUT(
-                 KC_P7,   KC_P8,   KC_P9, KC_PMNS, 
+                 KC_P7,   KC_P8,   KC_P9, KC_PMNS,
                  KC_P4,   KC_P5,   KC_P6, KC_PPLS,
         KC_MUTE, KC_P1,   KC_P2,   KC_P3, KC_TAB,
         KC_ESC,  KC_DEL, KC_P0, KC_PDOT, KC_PENT
     )
-        
+
 };
 
-void encoder_update_user(uint8_t index, bool clockwise) {
+bool encoder_update_user(uint8_t index, bool clockwise) {
   if (index == 0) { /* First encoder below the controller */
     if (clockwise) {
       tap_code(KC_VOLD); /*volume down*/
     } else {
       tap_code(KC_VOLU); /*volume up*/
     }
-  } else if (index == 1) { /* Second encoder */  
+  } else if (index == 1) { /* Second encoder */
     if (clockwise) {
       tap_code(KC_WH_U); /*mouse wheel up*/
     } else {
       tap_code(KC_WH_D); /*mouse wheel down*/
     }
   }
+    return true;
 }
diff --git a/keyboards/pabile/p20/ver1/keymaps/default/keymap.c b/keyboards/pabile/p20/ver1/keymaps/default/keymap.c
index 6b815e72e8..48b537b564 100644
--- a/keyboards/pabile/p20/ver1/keymaps/default/keymap.c
+++ b/keyboards/pabile/p20/ver1/keymaps/default/keymap.c
@@ -2,37 +2,38 @@
 
 const uint16_t PROGMEM keymaps[][MATRIX_ROWS][MATRIX_COLS] = {
     [0] = LAYOUT_ortho_5x4(
-        KC_PSLS, KC_PMNS, KC_PAST, KC_MPLY, 
-        KC_P7,   KC_P8,   KC_P9,   KC_PMNS, 
-        KC_P4,   KC_P5,   KC_P6,   KC_PPLS, 
-        KC_P1,   KC_P2,   KC_P3,   KC_TAB, 
+        KC_PSLS, KC_PMNS, KC_PAST, KC_MPLY,
+        KC_P7,   KC_P8,   KC_P9,   KC_PMNS,
+        KC_P4,   KC_P5,   KC_P6,   KC_PPLS,
+        KC_P1,   KC_P2,   KC_P3,   KC_TAB,
         LT(2,KC_P0), KC_PCMM, KC_PDOT, LT(1,KC_PENT)),
     [1] = LAYOUT_ortho_5x4(
-        KC_NLCK, KC_UNDS, KC_NO,   KC_NO, 
-        KC_AMPR, KC_ASTR, KC_LPRN, KC_EQL, 
-        KC_DLR,  KC_PERC, KC_CIRC, KC_NO, 
-        KC_EXLM, KC_AT,   KC_HASH, KC_NO, 
+        KC_NLCK, KC_UNDS, KC_NO,   KC_NO,
+        KC_AMPR, KC_ASTR, KC_LPRN, KC_EQL,
+        KC_DLR,  KC_PERC, KC_CIRC, KC_NO,
+        KC_EXLM, KC_AT,   KC_HASH, KC_NO,
         KC_RPRN, KC_NO,   KC_PSLS, KC_NO),
     [2] = LAYOUT_ortho_5x4(
-        KC_NLCK, KC_NO,   KC_NO,   KC_NO, 
-        KC_BTN1, KC_MS_U, KC_BTN2, KC_NO, 
-        KC_MS_L, KC_MS_D, KC_MS_R, KC_TAB, 
-        KC_WH_U, KC_NO,   KC_WH_D, KC_NO, 
+        KC_NLCK, KC_NO,   KC_NO,   KC_NO,
+        KC_BTN1, KC_MS_U, KC_BTN2, KC_NO,
+        KC_MS_L, KC_MS_D, KC_MS_R, KC_TAB,
+        KC_WH_U, KC_NO,   KC_WH_D, KC_NO,
         KC_NO,   KC_NO,   KC_DEL,  KC_ESC)
 };
 
-void encoder_update_user(uint8_t index, bool clockwise) {
+bool encoder_update_user(uint8_t index, bool clockwise) {
   if (index == 0) { /* First encoder */
     if (clockwise) {
       tap_code(KC_WH_U); /*mouse wheel up*/
     } else {
       tap_code(KC_WH_D); /*mouse wheel down */
     }
-  } else if (index == 1) { /* Second encoder */  
+  } else if (index == 1) { /* Second encoder */
     if (clockwise) {
       tap_code(KC_VOLD); /*volume down*/
     } else {
       tap_code(KC_VOLU); /*volume up*/
     }
   }
+    return true;
 }