Merge pull request +15144 from c9/vfs-fix-process-close

fix arguments for vfs process close event
This commit is contained in:
Harutyun Amirjanyan 2017-01-05 18:46:40 +04:00 committed by GitHub
commit 465daed6a3
2 changed files with 4 additions and 4 deletions

View File

@ -279,13 +279,13 @@ function Consumer() {
process.exited = true;
process.emit("exit", code, signal);
}
function onProcessClose(pid) {
function onProcessClose(pid, code, signal) {
var process = proxyProcesses[pid];
if (!process) return;
if (process.exited)
delete proxyProcesses[pid];
process.closed = true;
process.emit("close");
process.emit("close", code, signal);
}
function onPtyKill(pid){
var process = proxyProcesses[pid];

4
node_modules/vfs-socket/worker.js generated vendored
View File

@ -215,14 +215,14 @@ function Worker(vfs) {
delete processes[pid];
remote.onExit && remote.onExit(pid, code, signal);
});
process.on("close", function () {
process.on("close", function (code, signal) {
delete processes[pid];
if (!onlyPid) {
delete streams[process.stdout.id];
delete streams[process.stderr.id];
delete streams[process.stdin.id];
}
remote.onProcessClose && remote.onProcessClose(pid);
remote.onProcessClose && remote.onProcessClose(pid, code, signal);
});
process.kill = function(code, callback) {