#23 Moved logic for is localstorage function into anonymous function …

…which gets immediately called and set in a var for reuse
segmentio · Mar 11, 2021 · 1b6f52a · 1b6f52a
1 parent 67b4c8c
commit 1b6f52a
Showing 1 changed file with 6 additions and 6 deletions.
diff --git a/lib/engine.js b/lib/engine.js
@@ -33,7 +33,7 @@ var inMemoryStore = {
   }
 };
 
-function isSupportedNatively() {
+var isSupportedNatively = (function() {
   try {
     if (!window.localStorage) return false;
     var key = uuid();
@@ -47,17 +47,17 @@ function isSupportedNatively() {
     // Can throw if localStorage is disabled
     return false;
   }
-}
+}());
 
 function pickStorage() {
-  if (isSupportedNatively()) {
+  if (isSupportedNatively) {
     return window.localStorage;
   }
   // fall back to in-memory
   return inMemoryStore;
 }
 
-function isReadSupportedNatively() {
+var isReadSupportedNatively = (function() {
   try {
     if (!window.localStorage) return false;
     var key = window.localStorage.key(0);
@@ -69,10 +69,10 @@ function isReadSupportedNatively() {
     // Can throw if localStorage is disabled
     return false;
   }
-}
+}());
 
 function pickReclaimStorage() {
-  if (isReadSupportedNatively()) {
+  if (isSupportedNatively || isReadSupportedNatively) {
     return window.localStorage;
   }
   // fall back to in-memory