RFR: 8280818: Expand bug8033699.java to iterate over all LaFs
Harshitha Onkar
honkar at openjdk.org
Wed Mar 12 18:07:53 UTC 2025
On Wed, 12 Mar 2025 17:56:56 GMT, Alexey Ivanov <aivanov at openjdk.org> wrote:
>> test/jdk/javax/swing/JRadioButton/8033699/bug8033699.java line 121:
>>
>>> 119: if (mainFrame != null) {
>>> 120: mainFrame.dispose();
>>> 121: mainFrame = null;
>>
>> Assigning `mainFrame = null` is not required since we are disposing the frame at this point.
>
>> Assigning `mainFrame = null` is not required since we are disposing the frame at this point.
>
> It's required here, because the loop is repeated, and setting `mainFrame` to `null` prevents access to a disposed frame… likely from disposing of the frame again.
Missed seeing the finally block is within the LaF loop. Looks fine then.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/23964#discussion_r1992038099
More information about the client-libs-dev
mailing list