On Thu, 2 Jul 2026 15:40:20 GMT, Matthias Baesken <[email protected]> wrote:
>> src/java.desktop/windows/classes/sun/awt/windows/WPathGraphics.java line 332:
>>
>>> 330: } else {
>>> 331: return true;
>>> 332: }
>>
>> I'd use a direct `return`:
>>
>>
>>
>> return !(preferGDITextLayout ||
>> FontUtilities.textLayoutIsCompatible(font));
>>
>>
>> or after moving the logical negation into the parentheses
>>
>>
>> return !preferGDITextLayout &&
>> !FontUtilities.textLayoutIsCompatible(font);
>>
>>
>> However, the current with the `if` statement is perhaps clearer.
>
> Yeah it is a little bit clearer, so let's keep it this way !
Agreed.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/31725#discussion_r3519596958