Revert "try get_hold_on_other_key_press instead of achordion"

This reverts commit b2e5a3f645.

hold on other key press is suboptimal, too

ferris/config.h

@@ -42,8 +42,9 @@
 // Enable rapid switch from tap to hold, disables double tap hold auto-repeat.
 #define QUICK_TAP_TERM 0
 
-// we want that the thumb buttons are more reactive
-#define HOLD_ON_OTHER_KEY_PRESS_PER_KEY
+// try to be more permissive with holds, allows to trigger modifiers faster
+// achordion will avoid the worst
+#define PERMISSIVE_HOLD
 
 // caps word is great for defines
 #define BOTH_SHIFTS_TURNS_ON_CAPS_WORD

ferris/keymap.c

@@ -76,16 +76,17 @@ const uint16_t PROGMEM keymaps[][MATRIX_ROWS][MATRIX_COLS] = {
 
 };
 
-// Configure hold on other key press per key
-bool get_hold_on_other_key_press(uint16_t keycode, keyrecord_t *record) {
-    switch (keycode) {
-        // thumb buttons should be reactive
-        case MO(_SYM):
-        case LT(_NUM, KC_SPC):
-        case LT(_NAV, KC_BSPC):
-        case MO(_FN):
+// better handling of home row modifiers
+// see https://getreuer.info/posts/keyboards/achordion/index.html
+
+#include "achordion.h"
+#include "achordion.c"
+
+bool process_record_user(uint16_t keycode, keyrecord_t* record) {
+  if (!process_achordion(keycode, record)) { return false; }
   return true;
-        default:
-            return false;
 }
+
+void matrix_scan_user(void) {
+  achordion_task();
 }