On Thu, 18 Aug 2022 19:29:05 GMT, Roger Riggs <rri...@openjdk.org> wrote:
>> Revised test to check and close all of the streams used in the pipeline. >> >> The test now checks the content length and closes each stdout and stderr >> stream. >> >> The previous technique to redirect stderr to stdout was ineffective because >> the fd/stream was still created in the parent process. Created >> [8291986](https://bugs.openjdk.org/browse/JDK-8291986) >> ProcessBuilder.redirectErrorStream(true) leaves error stream available to >> investigate > > Roger Riggs has updated the pull request with a new target base due to a > merge or a rebase. The pull request now contains four commits: > > - Merge branch 'master' into 8291760-leaks-close > - Revised test to read, check, and close streams from all processes in the > pipeline. > If any unexpected output is found it is printed. > - Merge branch 'master' into 8291760-leaks-close > - 8291760: PipelineLeaksFD.java still fails: More or fewer pipes than > expected > Check the contents and close each stdout and stderr stream. Marked as reviewed by alanb (Reviewer). ------------- PR: https://git.openjdk.org/jdk/pull/9905