Remove redundant add ops in router

closes #1058
senchalabs · Oct 16, 2014 · 57cfe4f · 57cfe4f
1 parent e904e66
commit 57cfe4f
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/lib/proto.js b/lib/proto.js
@@ -101,8 +101,8 @@ app.use = function(route, fn){
 
 app.handle = function(req, res, out) {
   var stack = this.stack
-    , search = 1 + req.url.indexOf('?')
-    , pathlength = search ? search - 1 : req.url.length
+    , searchIndex = req.url.indexOf('?')
+    , pathlength = searchIndex !== -1 ? searchIndex : req.url.length
     , fqdn = 1 + req.url.substr(0, pathlength).indexOf('://')
     , protohost = fqdn ? req.url.substr(0, req.url.indexOf('/', 2 + fqdn)) : ''
     , removed = ''