Keep a local picsorter_deleted too.

Don't unpack it from localStorage every single time it is needed.
Read it in once and write it back when it changes.

Note: This would make updates from multiple tabs running picsorter
on the same domain clobber each other's updates.  So don't run
multiple picsorters.

diff --git a/picsorter.js b/picsorter.js
index c3af639441e865b47560208c5a605e0bb7437cc2..e44c22cad9abb5c79b5bdb1a23f9135b14a93fef 100644 (file)
--- a/picsorter.js
+++ b/picsorter.js
@@ -5,6 +5,7 @@ var zoom = "sm/";
 if (!("picsorter_deleted" in localStorage)) {
   localStorage.picsorter_deleted = JSON.stringify({});
 }
+var picsorter_deleted = JSON.parse(localStorage.picsorter_deleted);
 
 function setpic() {
   $("#pic").attr("src", zoom + exposure + "/" + files[input_index]);
@@ -19,22 +20,19 @@ function toggle_zoom() {
 }
 
 function move_to_nondeleted(direction) {
-  var deleted = JSON.parse(localStorage.picsorter_deleted);
   do {
     input_index += direction;
-  } while (files[input_index] in deleted);
+  } while (files[input_index] in picsorter_deleted);
 }
 
 function mark_deleted() {
-  var deleted = JSON.parse(localStorage.picsorter_deleted);
-  deleted[files[input_index]] = 1;
-  localStorage.picsorter_deleted = JSON.stringify(deleted);
+  picsorter_deleted[files[input_index]] = 1;
+  localStorage.picsorter_deleted = JSON.stringify(picsorter_deleted);
 }
 
 function mark_not_deleted() {
-  var deleted = JSON.parse(localStorage.picsorter_deleted);
-  delete deleted[files[input_index]];
-  localStorage.picsorter_deleted = JSON.stringify(deleted);
+  delete picsorter_deleted[files[input_index]];
+  localStorage.picsorter_deleted = JSON.stringify(picsorter_deleted);
 }
 
 $(function() {