RFR: 8319554: Select LogOutput* directly for stdout and stderr [v2]

[Xin Liu]

On Wed, 8 Nov 2023 19:16:05 GMT, Daniel D. Daugherty <dcubed at openjdk.org> wrote:

>> Sometimes in HotSpot you will see:
>> 
>> void *ptr = nullptr;
>> <some code>
>> if (nullptr == ptr) {
>>   <some code>
>> }
>> 
>> 
>> Some folks code variable checks that way because this:
>> 
>> `if (nullptr = ptr)`
>> 
>> will result in a compiler error whereas:
>> 
>> `if (ptr = nullptr)`
>> 
>> will not result in a compiler error. I've never heard that style called a Yoda condition before...
>
> I don't think I've seen it done with a function call return value though...

I read the wiki page. It mentioned that gcc -Wall will emit a warning when we use assignment as a condition. Defensive code style like `nullptr == ptr` may have been out of fashion nowadays.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/16543#discussion_r1387482107