extra keys: avoid scheduled executor leak

Leonid Pliushch · Leonid Pliushch · commit 0308d6a6ca74 · 2021-07-11T18:17:19.000+03:00
Under certain cases scheduled executor may leak causing repeatable input to stuck. Issue: #2156
diff --git a/app/src/main/java/com/termux/app/terminal/io/extrakeys/ExtraKeysView.java b/app/src/main/java/com/termux/app/terminal/io/extrakeys/ExtraKeysView.java
@@ -302,6 +302,10 @@ public void reload(ExtraKeysInfo infos) {
                         case MotionEvent.ACTION_DOWN:
                             longPressCount = 0;
                             v.setBackgroundColor(BUTTON_PRESSED_COLOR);
+                            if (scheduledExecutor != null) {
+                                scheduledExecutor.shutdownNow();
+                                scheduledExecutor = null;
+                            }
                             if (Arrays.asList("UP", "DOWN", "LEFT", "RIGHT", "BKSP", "DEL").contains(buttonInfo.getKey())) {
                                 // autorepeat
                                 scheduledExecutor = Executors.newSingleThreadScheduledExecutor();
