From 8fd04b1ac228392127df3ae084401f03a8d62306 Mon Sep 17 00:00:00 2001
From: Rongjian Zhang <pd4d10@gmail.com>
Date: Sat, 8 May 2021 17:09:18 +0800
Subject: [PATCH] http: refactor to remove redundant argument of
 _deferToConnect

---
 lib/_http_client.js | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

diff --git a/lib/_http_client.js b/lib/_http_client.js
index 4184191c0fbabc..0d7be7cbdde0a8 100644
--- a/lib/_http_client.js
+++ b/lib/_http_client.js
@@ -818,7 +818,7 @@ function onSocketNT(req, socket, err) {
 }
 
 ClientRequest.prototype._deferToConnect = _deferToConnect;
-function _deferToConnect(method, arguments_, cb) {
+function _deferToConnect(method, arguments_) {
   // This function is for calls that need to happen once the socket is
   // assigned to this request and writable. It's an important promisy
   // thing for all the socket calls that happen either now
@@ -828,9 +828,6 @@ function _deferToConnect(method, arguments_, cb) {
   const callSocketMethod = () => {
     if (method)
       ReflectApply(this.socket[method], this.socket, arguments_);
-
-    if (typeof cb === 'function')
-      cb();
   };
 
   const onSocket = () => {
@@ -846,7 +843,7 @@ function _deferToConnect(method, arguments_, cb) {
   } else {
     onSocket();
   }
-}
+};
 
 ClientRequest.prototype.setTimeout = function setTimeout(msecs, callback) {
   if (this._ended) {