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

repl: use promise#finally

Browse Source 
PR-URL: https://github.com/nodejs/node/pull/23971
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
This commit is contained in:
Weijia Wang 2018-10-30 16:33:00 +08:00 committed by Rich Trott
parent 7825045ee6
commit c1e670338b
1 changed files with 4 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
lib/repl.js
View File

@ -376,18 +376,8 @@ function REPLServer(prompt,
} }
promise.then((result) => { promise.then((result) => {
// Remove prioritized SIGINT listener if it was not called.
// TODO(TimothyGu): Use Promise.prototype.finally when it becomes
// available.
prioritizedSigintQueue.delete(sigintListener);
finishExecution(undefined, result); finishExecution(undefined, result);
unpause();
}, (err) => { }, (err) => {
// Remove prioritized SIGINT listener if it was not called.
prioritizedSigintQueue.delete(sigintListener);
unpause();
if (err && process.domain) { if (err && process.domain) {
debug('not recoverable, send to domain'); debug('not recoverable, send to domain');
process.domain.emit('error', err); process.domain.emit('error', err);
@ -395,6 +385,10 @@ function REPLServer(prompt,
return; return;
} }
finishExecution(err); finishExecution(err);
}).finally(() => {
// Remove prioritized SIGINT listener if it was not called.
prioritizedSigintQueue.delete(sigintListener);
unpause();
}); });
} }
} }