improve

trekjs · Apr 10, 2015 · 51b5e5e · 51b5e5e
1 parent 475decc
commit 51b5e5e
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 14 deletions.
diff --git a/lib/router.js b/lib/router.js
@@ -71,7 +71,7 @@ var Router = (function () {
 
         this.insert(method, path.substring(0, i), null, PNODE);
         // `/`
-        for (; i < l && path.charCodeAt(i) !== 47; i++) {}
+        for (; i < l && path.charCodeAt(i) !== 47; ++i) {}
 
         // new param key `$n`
         var param = '$' + count;
@@ -172,9 +172,9 @@ var Router = (function () {
     if (search.length === 0 || search === cn.prefix) {
       result[0] = cn.handler;
       if (cn.handler && cn.handler.keys) {
-        cn.handler.keys.forEach(function (a, i) {
-          params[i].name = a;
-        });
+        for (var i = 0, _l = cn.handler.keys.length; i < _l; ++i) {
+          params[i].name = cn.handler.keys[i];
+        }
       }
       return result;
     }
@@ -185,13 +185,13 @@ var Router = (function () {
       search = search.substring(l);
     }
 
-    for (var i = 0, k = cn.edges.length, e = undefined; i < k; i++) {
+    for (var i = 0, k = cn.edges.length, e = undefined; i < k; ++i) {
       e = cn.edges[i];
       var has = e.label === 58 ? PNODE : e.label === 42 ? CNODE : 0;
       if (has === PNODE) {
         l = search.length;
         // `/`
-        for (var j = 0; j < l && search.charCodeAt(j) !== 47; j++) {}
+        for (var j = 0; j < l && search.charCodeAt(j) !== 47; ++j) {}
 
         params[n] = {
           name: e.prefix.substring(1),
@@ -227,7 +227,7 @@ var Router = (function () {
 function lcp(a, b) {
   var i = 0;
   var max = Math.min(a.length, b.length);
-  for (; i < max && a.charCodeAt(i) === b.charCodeAt(i); i++) {}
+  for (; i < max && a.charCodeAt(i) === b.charCodeAt(i); ++i) {}
   return i;
 }
 

diff --git a/src/router.js b/src/router.js
@@ -68,7 +68,7 @@ class Router {
 
         this.insert(method, path.substring(0, i), null, PNODE);
         // `/`
-        for (; i < l && (path.charCodeAt(i) !== 47); i++) {}
+        for (; i < l && (path.charCodeAt(i) !== 47); ++i) {}
 
         // new param key `$n`
         let param = '$' + count;
@@ -163,9 +163,9 @@ class Router {
     if (search.length === 0 || search === cn.prefix) {
       result[0] = cn.handler;
       if (cn.handler && cn.handler.keys) {
-        cn.handler.keys.forEach((a, i) => {
-          params[i].name = a;
-        });
+        for (let i = 0, l = cn.handler.keys.length; i < l; ++i) {
+          params[i].name = cn.handler.keys[i];
+        }
       }
       return result;
     }
@@ -176,13 +176,13 @@ class Router {
       search = search.substring(l);
     }
 
-    for (let i = 0, k = cn.edges.length, e; i < k; i++) {
+    for (let i = 0, k = cn.edges.length, e; i < k; ++i) {
       e = cn.edges[i];
       let has = e.label === 58 ? PNODE : (e.label === 42 ? CNODE : 0);
       if (has === PNODE) {
         l = search.length;
         // `/`
-        for (var j = 0; j < l && (search.charCodeAt(j) !== 47); j++) {}
+        for (var j = 0; j < l && (search.charCodeAt(j) !== 47); ++j) {}
 
         params[n] = {
           name: e.prefix.substring(1),
@@ -217,7 +217,7 @@ class Router {
 function lcp(a, b) {
   let i = 0;
   let max = Math.min(a.length, b.length);
-  for (; i < max && (a.charCodeAt(i) === b.charCodeAt(i)); i++) {}
+  for (; i < max && (a.charCodeAt(i) === b.charCodeAt(i)); ++i) {}
   return i;
 }