[8] Review request for 7160951: [macosx] ActionListener called twice for JMenuItem using ScreenMenuBar

Sergey Bylokhov Sergey.Bylokhov at oracle.com
Fri Sep 7 12:53:24 PDT 2012


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 called 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/ 
> <http://cr.openjdk.java.net/%7Eleonidr/7160951/webrev.01/>
>
> Thanks,
> Leonid.


-- 
Best regards, Sergey.



More information about the macosx-port-dev mailing list