http: misc ClientRequest cleanup

PR-URL: #10654 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Fedor Indutny <fedor.indutny@gmail.com>
nodejs · Jan 13, 2017 · 2f77596 · aleclarson · Mar 19, 2018 · 2f77596
1 parent 15c6187
commit 2f77596
Showing 1 changed file with 2 additions and 5 deletions.
diff --git a/lib/_http_client.js b/lib/_http_client.js
@@ -97,9 +97,7 @@ function ClientRequest(options, cb) {
   var port = options.port = options.port || defaultPort || 80;
   var host = options.host = options.hostname || options.host || 'localhost';
 
-  if (options.setHost === undefined) {
-    var setHost = true;
-  }
+  var setHost = (options.setHost === undefined);
 
   self.socketPath = options.socketPath;
   self.timeout = options.timeout;
@@ -128,7 +126,7 @@ function ClientRequest(options, cb) {
   if (!headersArray) {
     if (options.headers) {
       var keys = Object.keys(options.headers);
-      for (var i = 0, l = keys.length; i < l; i++) {
+      for (var i = 0; i < keys.length; i++) {
         var key = keys[i];
         self.setHeader(key, options.headers[key]);
       }
@@ -154,7 +152,6 @@ function ClientRequest(options, cb) {
   }
 
   if (options.auth && !this.getHeader('Authorization')) {
-    //basic auth
     this.setHeader('Authorization', 'Basic ' +
                    Buffer.from(options.auth).toString('base64'));
   }