On Thu, 19 Sep 2024 16:12:45 GMT, Alexey Ivanov <[email protected]> wrote:
>> Abhishek Kumar has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> TestFrame var declared as local var
>
> test/jdk/java/awt/MenuItem/LotsOfMenuItemsTest.java line 75:
>
>> 73: @Override
>> 74: public void componentShown(ComponentEvent e) {
>> 75: TestFrame testFrame;
>
> I'd rather declare it before the loop — it's where `testFrame` is used.
Ok... Need re-review.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/21029#discussion_r1767121466
