1berry/nodejs
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

cluster: remove use of bind() in destroy()

Browse Source 
This commit replaces process.exit.bind() with an arrow function
in Worker.prototype.destroy().

PR-URL: https://github.com/nodejs/node/pull/6502
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
This commit is contained in:
yorkie 2016-05-01 22:55:05 +08:00 committed by cjihrig
parent db3db078f3
commit 4e905fa8d5
1 changed files with 6 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
lib/cluster.js
View File

@ -684,10 +684,12 @@ function workerInit() {
Worker.prototype.destroy = function() { Worker.prototype.destroy = function() {
this.exitedAfterDisconnect = true; this.exitedAfterDisconnect = true;
if (!this.isConnected()) process.exit(0); if (!this.isConnected()) {
var exit = process.exit.bind(null, 0); process.exit(0);
send({ act: 'exitedAfterDisconnect' }, () => process.disconnect()); } else {
process.once('disconnect', exit); send({ act: 'exitedAfterDisconnect' }, () => process.disconnect());
process.once('disconnect', () => process.exit(0));
}
}; };
function send(message, cb) { function send(message, cb) {