On Thu, 29 Feb 2024 11:02:35 GMT, Alexey Ivanov <[email protected]> wrote:

>> Abhishek Kumar has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   Review comment update
>
> src/java.desktop/windows/classes/com/sun/java/swing/plaf/windows/WindowsMenuBarUI.java
>  line 152:
> 
>> 150:                 //MenuElement[] path = new MenuElement[2];
>> 151:                 //path[0] = (MenuElement)menuBar;
>> 152:                 //path[1] = (MenuElement)menu;
> 
> Do not leave commented out code. The source control system is used to record 
> the history.
> 
> If you want to keep the old way, you can still do it by replacing one line
> 
> MenuElement[] path = {menuBar, menu};
> 
> inside `else` with these three lines.

Oh! my bad. I pushed it by mistake. Will update in next iteration.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/17961#discussion_r1507946680

Reply via email to