On Mon, 9 Feb 2026 22:57:43 GMT, Sergey Bylokhov <[email protected]> wrote:

>> Phil Race has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   8377191
>
> src/java.desktop/share/classes/java/awt/DefaultKeyboardFocusManager.java line 
> 265:
> 
>> 263:     static boolean sendMessage(Component target, AWTEvent e) {
>> 264:         e.isPosted = true;
>> 265:         final SentEvent se = new 
>> DefaultKeyboardFocusManagerSentEvent(e);
> 
> "final" for local variable is rarely used. Is it necessary here?

I did not add that. It was already there. See line 275 in the "original" code.
There may have been a reason but I don't see it. However it isn't a problem.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/29578#discussion_r2786009001

Reply via email to