From b2acb4a1ed4ae88bc17b74bcf7837b9953ff956c Mon Sep 17 00:00:00 2001 From: Aaron Culliney Date: Mon, 18 Apr 2016 21:30:01 -0700 Subject: [PATCH] Ensure we don't unnecessarily sync preferences to native --- .../main/java/org/deadc0de/apple2ix/Apple2Preferences.java | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/Android/app/src/main/java/org/deadc0de/apple2ix/Apple2Preferences.java b/Android/app/src/main/java/org/deadc0de/apple2ix/Apple2Preferences.java index 0454a01f..c60181db 100644 --- a/Android/app/src/main/java/org/deadc0de/apple2ix/Apple2Preferences.java +++ b/Android/app/src/main/java/org/deadc0de/apple2ix/Apple2Preferences.java @@ -212,8 +212,10 @@ public class Apple2Preferences { public static void save(Apple2Activity activity) { - // bespoke reset temporary values + // bespoke reset temporary values... + boolean wasDirty = sNativeIsDirty.get(); Apple2Preferences.setJSONPref(Apple2Preferences.PREF_DOMAIN_TOUCHSCREEN, Apple2Preferences.PREF_CALIBRATING, false); + sNativeIsDirty.set(wasDirty); File prefsFile = new File(getPrefsFile(activity)); String jsonString = null; @@ -243,7 +245,10 @@ public class Apple2Preferences { public static void sync(Apple2Activity activity, String domain) { save(activity); if (sNativeIsDirty.getAndSet(false)) { + Log.v(TAG, "Syncing prefs to native..."); nativePrefsSync(domain); + } else { + Log.v(TAG, "No changes, not syncing prefs..."); } }