<i18n dev> RFR: 8377191: Remove AppContext from KeyboardFocusManager [v2]

Phil Race prr at openjdk.org
Wed Feb 11 21:02:58 UTC 2026


On Mon, 9 Feb 2026 22:57:43 GMT, Sergey Bylokhov <serb at openjdk.org> 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


More information about the i18n-dev mailing list