On Mon, 18 Nov 2024 19:55:58 GMT, Phil Race <p...@openjdk.org> wrote:
>> src/java.desktop/windows/classes/com/sun/java/swing/plaf/windows/XPStyle.java
>> line 134:
>>
>>> 132: }
>>> 133: if (themeActive.booleanValue()) {
>>> 134: String propertyAction =
>>> System.getProperty("swing.noxp");
>>
>> The variable name is now confusing… it's not an action anymore. Alternative,
>> `getProperty` can be inlined in the `if` statement.
>
> A goal here is to MINIMISE optional refactoring, this is borderline. I'll
> leave it up to Prasanta.
SInce there is no more change to be done, I will skip this..It can be handled
separately in a cleanup later on..
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/22083#discussion_r1847578343
