RFR: 6318027: BasicScrollBarUI does not disable timer when enclosing frame is disabled. [v5]

Abhishek Kumar abhiscxk at openjdk.org
Tue Aug 20 06:47:52 UTC 2024


On Tue, 20 Aug 2024 06:34:47 GMT, Prasanta Sadhukhan <psadhukhan at openjdk.org> wrote:

>> src/java.desktop/macosx/classes/com/apple/laf/AquaScrollBarUI.java line 534:
>> 
>>> 532:                 if (parent instanceof javax.swing.JFrame par) {
>>> 533:                     if (!par.isEnabled()) {
>>> 534:                         ((Timer)e.getSource()).stop();
>> 
>> Suggestion:
>> 
>>                         ((Timer) e.getSource()).stop();
>
> same spacing issue in other places too as mentioned above...

Ok... as you are changing the file I guess you can expand the wild imports, It's up to you.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/20346#discussion_r1722772262


More information about the client-libs-dev mailing list