Skip to content

Commit

Permalink
net: refactor onSlaveClose in Server.close
Browse files Browse the repository at this point in the history
Refactors onSlaveClose in Server.close to be an arrow function,
removes need for `self = this` and moves it down to make code
more readable.

PR-URL: #12334
  • Loading branch information
claudiorodriguez committed Apr 11, 2017
1 parent 569f988 commit 09c972a
Showing 1 changed file with 6 additions and 8 deletions.
14 changes: 6 additions & 8 deletions lib/net.js
Original file line number Diff line number Diff line change
@@ -1539,13 +1539,6 @@ Server.prototype.getConnections = function(cb) {


Server.prototype.close = function(cb) {
function onSlaveClose() {
if (--left !== 0) return;

self._connections = 0;
self._emitCloseIfDrained();
}

if (typeof cb === 'function') {
if (!this._handle) {
this.once('close', function close() {
@@ -1562,8 +1555,13 @@ Server.prototype.close = function(cb) {
}

if (this._usingSlaves) {
var self = this;
var left = this._slaves.length;
const onSlaveClose = () => {
if (--left !== 0) return;

this._connections = 0;
this._emitCloseIfDrained();
};

// Increment connections to be sure that, even if all sockets will be closed
// during polling of slaves, `close` event will be emitted only once.

0 comments on commit 09c972a

Please sign in to comment.