Skip to content

Commit

Permalink
child_process: remove unnecessary use of inner state
Browse files Browse the repository at this point in the history
PR-URL: #29358
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
  • Loading branch information
ckarande authored and Trott committed Jan 12, 2020
1 parent 4de31d5 commit f9c16b8
Showing 1 changed file with 1 addition and 3 deletions.
4 changes: 1 addition & 3 deletions lib/internal/child_process.js
Original file line number Diff line number Diff line change
Expand Up @@ -301,9 +301,7 @@ function flushStdio(subprocess) {
// TODO(addaleax): This doesn't necessarily account for all the ways in
// which data can be read from a stream, e.g. being consumed on the
// native layer directly as a StreamBase.
if (!stream || !stream.readable ||
stream._readableState.readableListening ||
stream[kIsUsedAsStdio]) {
if (!stream || !stream.readable || stream[kIsUsedAsStdio]) {
continue;
}
stream.resume();
Expand Down

0 comments on commit f9c16b8

Please sign in to comment.