<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