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