net: don't create unnecessary closure

Pass arguments to fireErrorCallbacks() explicitly. Saves allocation an unnecessary closure context. PR-URL: #12342 Reviewed-By: Benjamin Gruenbaum <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Evan Lucas <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Luigi Pinca <[email protected]>

Author: bnoordhuis

lib/net.js

@@ -476,7 +476,7 @@ Socket.prototype.destroySoon = function(){
 Socket.prototype._destroy=function(exception,cb){
 debug('destroy');
 
-functionfireErrorCallbacks(self){
+functionfireErrorCallbacks(self,exception,cb){
 if(cb)cb(exception);
 if(exception&&!self._writableState.errorEmitted){
 process.nextTick(emitErrorNT,self,exception);
@@ -486,7 +486,7 @@ Socket.prototype._destroy = function(exception, cb){
 
 if(this.destroyed){
 debug('already destroyed, fire error callbacks');
-fireErrorCallbacks(this);
+fireErrorCallbacks(this,exception,cb);
 return;
 }
 
@@ -518,7 +518,7 @@ Socket.prototype._destroy = function(exception, cb){
 // to make it re-entrance safe in case Socket.prototype.destroy()
 // is called within callbacks
 this.destroyed=true;
-fireErrorCallbacks(this);
+fireErrorCallbacks(this,exception,cb);
 
 if(this._server){
 COUNTER_NET_SERVER_CONNECTION_CLOSE(this);