optimize source number picker <SCRIPT> IO to start sooner

bulk88 · Apr 5, 2024 · 17a4168 · 17a4168
1 parent 8ec965a
commit 17a4168
Show file tree

Hide file tree

Showing 3 changed files with 23 additions and 17 deletions.
diff --git a/client.js b/client.js
@@ -1188,10 +1188,17 @@ function getCarrierForLinked(numIdx, node, nextCarrierCB) {
   return function () {
     carrierUpdate = function (s) {
       //this obj is window obj
-      var i = 1, actInfo_rPid = s.match(/(CINGULAR)|(CELLCO PARTNERSHIP)|(OMNIPOINT|T-MOBILE USA|METROPCS)|(BANDWIDTH.COM)|(SPRINT SPECTRUM)|(Inteliquent)|(PEERLESS NETWORK OF NEW YORK)/);
+      var actInfo_rPid;
+      var oldCarrierUpdate = carrierUpdate;
+      var i = oldCarrierUpdate.next;
+      //free memory of window. global 1x use function
+      carrierUpdate = null;
+      i && i(); //run next carrier getter ASAP for I/O, note the JSONP api only allows 1 cb at a time
+
+      actInfo_rPid = s.match(/(CINGULAR)|(CELLCO PARTNERSHIP)|(OMNIPOINT|T-MOBILE USA|METROPCS)|(BANDWIDTH.COM)|(SPRINT SPECTRUM)|(Inteliquent)|(PEERLESS NETWORK OF NEW YORK)/);
       if(actInfo_rPid) {
         //7 total
-        for (; i < 8; i++) {
+        for (i = 1; i < 8; i++) {
           if (actInfo_rPid[i]) {
             s = [,
               'ATTW',
@@ -1207,14 +1214,10 @@ function getCarrierForLinked(numIdx, node, nextCarrierCB) {
         }
       }
       actInfo_rPid = lazySignedInDIDLinkedPhone();
-      carrierUpdate.wvnode.nodeValue = actInfo_rPid[ACTNUM_LINKED()][numIdx][LINKED_CARRIER()] = "\xa0"+s;
+      oldCarrierUpdate.wvnode.nodeValue = actInfo_rPid[ACTNUM_LINKED()][numIdx][LINKED_CARRIER()] = "\xa0"+s;
       setDIDLinkedPhone(actInfo_rPid);
       //rmv script tag, they build up if SPA
-      (i=carrierUpdate.script).parentNode.removeChild(i);
-      //free memory of window. global 1x use function
-      i = carrierUpdate.next;
-      carrierUpdate = null; //wipe global
-      i && i(); //run next carrier getter, note the JSONP api only allows 1 cb at a time
+      (i=oldCarrierUpdate.script).parentNode.removeChild(i);
     }
     carrierUpdate.wvnode = node;
     carrierUpdate.next = nextCarrierCB;

diff --git a/docs/ac.appcache b/docs/ac.appcache
@@ -1,5 +1,5 @@
 CACHE MANIFEST
-# v Fri Apr  5 01:09:49 2024
+# v Fri Apr  5 04:19:53 2024
 
 CACHE:
 index.html

diff --git a/docs/client.js b/docs/client.js
@@ -1188,10 +1188,17 @@ function getCarrierForLinked(numIdx, node, nextCarrierCB) {
   return function () {
     carrierUpdate = function (s) {
       //this obj is window obj
-      var i = 1, actInfo_rPid = s.match(/(CINGULAR)|(CELLCO PARTNERSHIP)|(OMNIPOINT|T-MOBILE USA|METROPCS)|(BANDWIDTH.COM)|(SPRINT SPECTRUM)|(Inteliquent)|(PEERLESS NETWORK OF NEW YORK)/);
+      var actInfo_rPid;
+      var oldCarrierUpdate = carrierUpdate;
+      var i = oldCarrierUpdate.next;
+      //free memory of window. global 1x use function
+      carrierUpdate = null;
+      i && i(); //run next carrier getter ASAP for I/O, note the JSONP api only allows 1 cb at a time
+
+      actInfo_rPid = s.match(/(CINGULAR)|(CELLCO PARTNERSHIP)|(OMNIPOINT|T-MOBILE USA|METROPCS)|(BANDWIDTH.COM)|(SPRINT SPECTRUM)|(Inteliquent)|(PEERLESS NETWORK OF NEW YORK)/);
       if(actInfo_rPid) {
         //7 total
-        for (; i < 8; i++) {
+        for (i = 1; i < 8; i++) {
           if (actInfo_rPid[i]) {
             s = [,
               'ATTW',
@@ -1207,14 +1214,10 @@ function getCarrierForLinked(numIdx, node, nextCarrierCB) {
         }
       }
       actInfo_rPid = lazySignedInDIDLinkedPhone();
-      carrierUpdate.wvnode.nodeValue = actInfo_rPid[ACTNUM_LINKED()][numIdx][LINKED_CARRIER()] = "\xa0"+s;
+      oldCarrierUpdate.wvnode.nodeValue = actInfo_rPid[ACTNUM_LINKED()][numIdx][LINKED_CARRIER()] = "\xa0"+s;
       setDIDLinkedPhone(actInfo_rPid);
       //rmv script tag, they build up if SPA
-      (i=carrierUpdate.script).parentNode.removeChild(i);
-      //free memory of window. global 1x use function
-      i = carrierUpdate.next;
-      carrierUpdate = null; //wipe global
-      i && i(); //run next carrier getter, note the JSONP api only allows 1 cb at a time
+      (i=oldCarrierUpdate.script).parentNode.removeChild(i);
     }
     carrierUpdate.wvnode = node;
     carrierUpdate.next = nextCarrierCB;