Request for review: 7144063: [macosx] Swing JMenu mnemonic doesn't work; hint misleading; cross symbol typed
Scott Kovatch
scott.kovatch at oracle.com
Tue Feb 14 08:24:27 PST 2012
Alexander,
I recall writing that code originally a very long time ago, so my memory is hazy. Maybe Mike can do a 'cvs blame' on the old Apple codebase for the original reason I added that check.
I seem to recall it was for handling some of the less common input methods like Bulgarian or Cherokee - Nation that can trigger multiple characters on a single keystroke.
This is not to say your change is wrong, but it would be good to check the test case for why I added that check in the first place.
-- Scott
On Feb 14, 2012, at 7:50 AM, Alexander Zuev wrote:
> Hello,
>
> please review my fix for bug 7144063: [macosx] Swing JMenu mnemonic doesn't work; hint misleading; cross symbol typed
>
> Bug description: http://bugs.sun.com/view_bug.do?bug_id=7144063
>
> Webrev for proposed change: http://cr.openjdk.java.net/~kizune/7144063/webrev.00/
>
> The idea of the fix is to call the java handler for creation of InputMethod events
> for all the complex symbols instead of just for symbols with length of 3 bytes or more.
>
> With best regards,
> Alexander Zuev
More information about the macosx-port-dev
mailing list