From f80b873866f07fe1f37a9de0dfaeaf7aecdb1feb Mon Sep 17 00:00:00 2001
From: Andreas Shimokawa <shimokawa@fsfe.org>
Date: Wed, 14 Jun 2023 16:44:51 +0200
Subject: [PATCH] Prefs: make allow high MTU default to true.

It is 2023, it should work, if not it can be turned off.

For newer devices with big firmwares and apps (like Mi Band 7), people just wonder how slow Gadgetbridge installs stuff...
---
 .../service/devices/banglejs/BangleJSDeviceSupport.java         | 2 +-
 .../gadgetbridge/service/devices/huami/HuamiSupport.java        | 2 +-
 app/src/main/res/xml/devicesettings_high_mtu.xml                | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/service/devices/banglejs/BangleJSDeviceSupport.java b/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/service/devices/banglejs/BangleJSDeviceSupport.java
index 591ed6835..a7ebf21d2 100644
--- a/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/service/devices/banglejs/BangleJSDeviceSupport.java
+++ b/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/service/devices/banglejs/BangleJSDeviceSupport.java
@@ -317,7 +317,7 @@ public class BangleJSDeviceSupport extends AbstractBTLEDeviceSupport {
         builder.notify(rxCharacteristic, true);
 
         Prefs devicePrefs = new Prefs(GBApplication.getDeviceSpecificSharedPrefs(gbDevice.getAddress()));
-        allowHighMTU = devicePrefs.getBoolean(PREF_ALLOW_HIGH_MTU, false);
+        allowHighMTU = devicePrefs.getBoolean(PREF_ALLOW_HIGH_MTU, true);
 
         uartTx(builder, " \u0003"); // clear active line
 
diff --git a/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/service/devices/huami/HuamiSupport.java b/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/service/devices/huami/HuamiSupport.java
index e137cc0ba..759d95e5d 100644
--- a/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/service/devices/huami/HuamiSupport.java
+++ b/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/service/devices/huami/HuamiSupport.java
@@ -4142,7 +4142,7 @@ public abstract class HuamiSupport extends AbstractBTLEDeviceSupport implements
 
     protected void setMtu(final int mtu) {
         final Prefs prefs = getDevicePrefs();
-        if (!prefs.getBoolean(PREF_ALLOW_HIGH_MTU, false)) {
+        if (!prefs.getBoolean(PREF_ALLOW_HIGH_MTU, true)) {
             LOG.warn("High MTU is not allowed, ignoring");
             return;
         }
diff --git a/app/src/main/res/xml/devicesettings_high_mtu.xml b/app/src/main/res/xml/devicesettings_high_mtu.xml
index 68a982397..b721b59c8 100644
--- a/app/src/main/res/xml/devicesettings_high_mtu.xml
+++ b/app/src/main/res/xml/devicesettings_high_mtu.xml
@@ -2,7 +2,7 @@
 <androidx.preference.PreferenceScreen xmlns:android="http://schemas.android.com/apk/res/android">
     <SwitchPreference
         android:icon="@drawable/ic_mtu"
-        android:defaultValue="false"
+        android:defaultValue="true"
         android:key="allow_high_mtu"
         android:summary="@string/pref_summary_allow_high_mtu"
         android:title="@string/pref_title_allow_high_mtu" />