RFR: 8233725: ProcessTools.startProcess() has output issues when using an OutputAnalyzer at the same time [v2]
Leonid Mesnik
lmesnik at openjdk.org
Wed Apr 26 14:40:24 UTC 2023
On Wed, 26 Apr 2023 09:38:45 GMT, Serguei Spitsyn <sspitsyn at openjdk.org> wrote:
>> Right. From the API caller's POV, it is asking for InputStreams that it can use to read the process' stdout or stderr streams.
>
> Okay, thanks.
> I'm thinking about simple/short comments on this to make it clear this naming mismatch is intentional.
There is a comment in line 156 explaining the purpose of these streams.
Also, I renamed out/err names to stdOut/stdErr to make it clearer.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/13594#discussion_r1177949748
More information about the serviceability-dev
mailing list