lib: make lowerProto scope more clear

gengjiawen · gengjiawen · commit d768c5c6f2d5 · 2019-03-10T13:42:08.000+08:00
diff --git a/lib/url.js b/lib/url.js
@@ -250,10 +250,11 @@ Url.prototype.parse = function parse(url, parseQueryString, slashesDenoteHost) {
     }
   }
 
-  var proto = protocolPattern.exec(rest);
+  let proto = protocolPattern.exec(rest);
+  let lowerProto;
   if (proto) {
     proto = proto[0];
-    var lowerProto = proto.toLowerCase();
+    lowerProto = proto.toLowerCase();
     this.protocol = lowerProto;
     rest = rest.slice(proto.length);
   }

Original file line number	Diff line number	Diff line change
`@@ -250,10 +250,11 @@ Url.prototype.parse = function parse(url, parseQueryString, slashesDenoteHost) {`
`250`	`250`	`}`
`251`	`251`	`}`
`252`	`252`
`253`		`- var proto = protocolPattern.exec(rest);`
	`253`	`+ let proto = protocolPattern.exec(rest);`
	`254`	`+ let lowerProto;`
`254`	`255`	`if (proto) {`
`255`	`256`	`proto = proto[0];`
`256`		`- var lowerProto = proto.toLowerCase();`
	`257`	`+ lowerProto = proto.toLowerCase();`
`257`	`258`	`this.protocol = lowerProto;`
`258`	`259`	`rest = rest.slice(proto.length);`
`259`	`260`	`}`