RFR: JDK-8292351: tty should always live [v5]
Thomas Stuefe
stuefe at openjdk.org
Mon Aug 22 08:06:26 UTC 2022
On Mon, 22 Aug 2022 07:55:17 GMT, David Holmes <dholmes at openjdk.org> wrote:
>> Thomas Stuefe has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Compromise
>
> src/hotspot/share/utilities/ostream.cpp line 982:
>
>> 980: delete tty;
>> 981: }
>> 982: tty = DisplayVMOutputToStderr ? fdStream::stdout_stream() : fdStream::stderr_stream();
>
> Shouldn't this be something like:
>
> tmp = tty;
> tty = ...
> delete tmp;
It probably does not make much difference since the swap-over is not atomic anyway, but I can change it.
-------------
PR: https://git.openjdk.org/jdk/pull/9874
More information about the hotspot-dev
mailing list