On Tue, 27 Jan 2026 13:17:26 GMT, Alexey Ivanov <[email protected]> wrote:

>> Prasanta Sadhukhan has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   Invoker null check
>
> src/java.desktop/share/classes/javax/swing/JPopupMenu.java line 970:
> 
>> 968:         invalidate();
>> 969: 
>> 970:     }
> 
> Does this newly added blank line right before the closing brace of the method 
> serve any purpose? I'd rather not add it.

What happens if the new invoker is null, the old invoker is not null and the UI 
is null? Will we continue to receive events from the old invoker? Or this 
combination is not possible?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/29377#discussion_r2739728205

Reply via email to