Added: Add support for ~/.termux/termux.float.properties

app/src/main/java/com/termux/app/settings/properties/TermuxAppSharedProperties.java

@@ -9,6 +9,7 @@ import com.termux.shared.terminal.io.extrakeys.ExtraKeysInfo;
 import com.termux.shared.logger.Logger;
 import com.termux.shared.settings.properties.TermuxPropertyConstants;
 import com.termux.shared.settings.properties.TermuxSharedProperties;
+import com.termux.shared.termux.TermuxConstants;
 
 import org.json.JSONException;
 
@@ -26,7 +27,8 @@ public class TermuxAppSharedProperties extends TermuxSharedProperties {
     private static final String LOG_TAG = "TermuxAppSharedProperties";
 
     public TermuxAppSharedProperties(@Nonnull Context context) {
-        super(context);
+        super(context, TermuxConstants.TERMUX_APP_NAME, TermuxPropertyConstants.getTermuxPropertiesFile(),
+            TermuxPropertyConstants.TERMUX_PROPERTIES_LIST, new SharedPropertiesParserClient());
     }
 
     /**
@@ -110,7 +112,8 @@ public class TermuxAppSharedProperties extends TermuxSharedProperties {
      * Load the {@link TermuxPropertyConstants#KEY_TERMINAL_TRANSCRIPT_ROWS} value from termux properties file on disk.
      */
     public static int getTerminalTranscriptRows(Context context) {
-        return  (int) TermuxSharedProperties.getInternalPropertyValue(context, TermuxPropertyConstants.KEY_TERMINAL_TRANSCRIPT_ROWS);
+        return  (int) TermuxSharedProperties.getInternalPropertyValue(context, TermuxPropertyConstants.getTermuxPropertiesFile(),
+            TermuxPropertyConstants.KEY_TERMINAL_TRANSCRIPT_ROWS, new SharedPropertiesParserClient());
     }
 
 }

app/src/main/java/com/termux/app/terminal/TermuxTerminalSessionClient.java

@@ -20,7 +20,7 @@ import com.termux.shared.terminal.TermuxTerminalSessionClientBase;
 import com.termux.shared.termux.TermuxConstants;
 import com.termux.app.TermuxService;
 import com.termux.shared.settings.properties.TermuxPropertyConstants;
-import com.termux.app.terminal.io.BellHandler;
+import com.termux.shared.terminal.io.BellHandler;
 import com.termux.shared.logger.Logger;
 import com.termux.terminal.TerminalColors;
 import com.termux.terminal.TerminalSession;

app/src/main/java/com/termux/app/terminal/TermuxTerminalViewClient.java

@@ -140,7 +140,7 @@ public class TermuxTerminalViewClient extends TermuxTerminalViewClientBase {
     }
 
     /**
-     * Should be called when {@link com.termux.view.TerminalView#mEmulator}
+     * Should be called when {@link com.termux.view.TerminalView#mEmulator} is set
      */
     @Override
     public void onEmulatorSet() {

app/src/main/java/com/termux/app/terminal/io/BellHandler.java

@@ -1,79 +0,0 @@
-package com.termux.app.terminal.io;
-
-import android.content.Context;
-import android.os.Build;
-import android.os.Handler;
-import android.os.Looper;
-import android.os.SystemClock;
-import android.os.VibrationEffect;
-import android.os.Vibrator;
-
-import com.termux.shared.logger.Logger;
-
-public class BellHandler {
-    private static BellHandler instance = null;
-    private static final Object lock = new Object();
-
-    private static final String LOG_TAG = "BellHandler";
-
-    public static BellHandler getInstance(Context context) {
-        if (instance == null) {
-            synchronized (lock) {
-                if (instance == null) {
-                    instance = new BellHandler((Vibrator) context.getApplicationContext().getSystemService(Context.VIBRATOR_SERVICE));
-                }
-            }
-        }
-
-        return instance;
-    }
-
-    private static final long DURATION = 50;
-    private static final long MIN_PAUSE = 3 * DURATION;
-
-    private final Handler handler = new Handler(Looper.getMainLooper());
-    private long lastBell = 0;
-    private final Runnable bellRunnable;
-
-    private BellHandler(final Vibrator vibrator) {
-        bellRunnable = new Runnable() {
-            @Override
-            public void run() {
-                if (vibrator != null) {
-                    try {
-                        if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.O) {
-                            vibrator.vibrate(VibrationEffect.createOneShot(DURATION, VibrationEffect.DEFAULT_AMPLITUDE));
-                        } else {
-                            vibrator.vibrate(DURATION);
-                        }
-                    } catch (Exception e) {
-                        // Issue on samsung devices on android 8
-                        // java.lang.NullPointerException: Attempt to read from field 'android.os.VibrationEffect com.android.server.VibratorService$Vibration.mEffect' on a null object reference
-                        Logger.logStackTraceWithMessage(LOG_TAG, "Failed to run vibrator", e);
-                    }
-                }
-            }
-        };
-    }
-
-    public synchronized void doBell() {
-        long now = now();
-        long timeSinceLastBell = now - lastBell;
-
-        if (timeSinceLastBell < 0) {
-            // there is a next bell pending; don't schedule another one
-        } else if (timeSinceLastBell < MIN_PAUSE) {
-            // there was a bell recently, schedule the next one
-            handler.postDelayed(bellRunnable, MIN_PAUSE - timeSinceLastBell);
-            lastBell = lastBell + MIN_PAUSE;
-        } else {
-            // the last bell was long ago, do it now
-            bellRunnable.run();
-            lastBell = now;
-        }
-    }
-
-    private long now() {
-        return SystemClock.uptimeMillis();
-    }
-}