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