extra keys: avoid scheduled executor leak

Under certain cases scheduled executor may leak causing repeatable input to stuck. Issue: #2156
termux · Jul 11, 2021 · 0308d6a · 0308d6a
1 parent 1b62f7c
commit 0308d6a
Showing 1 changed file with 4 additions and 0 deletions.
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();