From d09eb9c6b26aeeed0cb6d2e8a1937286eccdede0 Mon Sep 17 00:00:00 2001 From: Pedro Victor Date: Sat, 29 Oct 2016 13:39:53 -0200 Subject: [PATCH] net: name anonymous functions MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit the changes are related https://github.com/nodejs/node/issues/8913 regarding the naming of just the inline anonymous functions that are not assigned to a variable PR-URL: https://github.com/nodejs/node/pull/9357 Reviewed-By: Rich Trott Reviewed-By: James M Snell Reviewed-By: Michaƫl Zasso Reviewed-By: Ilkka Myller Reviewed-By: Matteo Collina Reviewed-By: Roman Reiss --- lib/net.js | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/lib/net.js b/lib/net.js index 628908614fbd61..ce60d2f3521317 100644 --- a/lib/net.js +++ b/lib/net.js @@ -269,7 +269,7 @@ function onSocketEnd() { this.readable = false; maybeDestroy(this); } else { - this.once('end', function() { + this.once('end', function end() { this.readable = false; maybeDestroy(this); }); @@ -669,7 +669,7 @@ Socket.prototype._writeGeneric = function(writev, data, encoding, cb) { if (this.connecting) { this._pendingData = data; this._pendingEncoding = encoding; - this.once('connect', function() { + this.once('connect', function connect() { this._writeGeneric(writev, data, encoding, cb); }); return; @@ -991,7 +991,7 @@ function lookupAndConnect(self, options) { debug('connect: dns options', dnsopts); self._host = host; var lookup = options.lookup || dns.lookup; - lookup(host, dnsopts, function(err, ip, addressType) { + lookup(host, dnsopts, function emitLookup(err, ip, addressType) { self.emit('lookup', err, ip, addressType, host); // It's possible we were destroyed while looking this up. @@ -1389,7 +1389,7 @@ Server.prototype.listen = function() { }; function lookupAndListen(self, port, address, backlog, exclusive) { - require('dns').lookup(address, function(err, ip, addressType) { + require('dns').lookup(address, function doListening(err, ip, addressType) { if (err) { self.emit('error', err); } else { @@ -1494,7 +1494,7 @@ Server.prototype.close = function(cb) { if (typeof cb === 'function') { if (!this._handle) { - this.once('close', function() { + this.once('close', function close() { cb(new Error('Not running')); }); } else {