I dunno. I just did some cleanup of existing code, I didn't dig deeper.  I'll 
investigate it. 

On Sep 7, 2012, at 11:53 PM, Sergey Bylokhov <[email protected]> wrote:

> Hi, Leonid.
> Why we cannot use javaExtMask everywhere in CMenuItem.java.setLabel(...) && 
> CMenuItem.m ?
> 
> 07.09.2012 22:41, Leonid Romanov wrote:
>> Hi,
>> Please review a fix for 7160951: [macosx] ActionListener calle d twice for 
>> JMenuItem using ScreenMenuBar.  Although a number of files have been 
>> changed, only CMenuItem.m changes constitutes the fix, the rest of the 
>> changes is a bit of code cleanup.
>> The fix has been contributed by Marco Dinacci. Thanks!
>> 
>> Bug: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7160951
>> Webrev: http://cr.openjdk.java.net/~leonidr/7160951/webrev.01/
>> 
>> Thanks,
>> Leonid.
> 
> 
> -- 
> Best regards, Sergey. 

Reply via email to