From: Nathaniel Wesley Filardo <nwf@cs.jhu.edu>
Date: Sat, 17 Sep 2016 19:48:46 +0000 (-0400)
Subject: Grid vis: mutate debug pref in onClick
X-Git-Url: https://hydra-www.ietfng.org/gitweb/?a=commitdiff_plain;p=acmetensortoys-watchviz

Grid vis: mutate debug pref in onClick
---

diff --git a/vizlib/src/main/java/com/acmetensortoys/watchviz/vizlib/rendering/Grid.java b/vizlib/src/main/java/com/acmetensortoys/watchviz/vizlib/rendering/Grid.java
index ad25435..ad136d8 100644
--- a/vizlib/src/main/java/com/acmetensortoys/watchviz/vizlib/rendering/Grid.java
+++ b/vizlib/src/main/java/com/acmetensortoys/watchviz/vizlib/rendering/Grid.java
@@ -16,6 +16,7 @@ public final class Grid extends Rendering {
     private final Paint p = new Paint();
     private final Paint dbp = new Paint();
     private float[] hsv = new float[]{0.0f, 1.0f, 1.0f};
+    private final SharedPreferences lsp;
 
     // 2^7 == 128 frames, at 512 samples per frame and 11025 KHz, this works out
     // to six seconds, which seems fine.
@@ -29,8 +30,8 @@ public final class Grid extends Rendering {
         public void onSharedPreferenceChanged(SharedPreferences sp, String key) {
             Log.d("GridOSPCL", key);
             switch(key) {
-                case "debug": debugByPref(sp); break;
-                case "cycle": cycleByPref(sp); break;
+                case "debug": debugByPref(); break;
+                case "cycle": cycleByPref(); break;
             }
         }
     };
@@ -41,22 +42,24 @@ public final class Grid extends Rendering {
         dbp.setColor(Color.WHITE);
         dbp.setTypeface(Typeface.MONOSPACE);
 
+        this.lsp = lsp;
         lsp.registerOnSharedPreferenceChangeListener(ospcl);
-        debugByPref(lsp);
-        cycleByPref(lsp);
+        debugByPref();
+        cycleByPref();
     }
 
-    private void debugByPref(SharedPreferences sp) {
-        doDebug = sp.getBoolean("debug", false);
+    private void debugByPref() {
+        doDebug = lsp.getBoolean("debug", false);
     }
-    private void cycleByPref(SharedPreferences sp) {
-        doCycle = sp.getBoolean("cycle", true);
+    private void cycleByPref() {
+        doCycle = lsp.getBoolean("cycle", true);
     }
 
 
     @Override
     public void onClick() {
         doDebug = !doDebug;
+        lsp.edit().putBoolean("debug", doDebug).apply();
     }
 
     @Override