<AWT Dev> [9] Review Request: 8037096 [macosx] Refactor native menu code

Sergey Bylokhov Sergey.Bylokhov at oracle.com
Sun Mar 30 23:58:51 UTC 2014


Hi, Petr.
CMenu.m: In a few places you inline the call to the createCMenu, 
probably the method is unnecessary now? or inlining is not necessary?

On 3/27/14 1:53 PM, Petr Pchelko wrote:
> Hello, AWT Team.
>
> Please review the fix for the issue: 
> https://bugs.openjdk.java.net/browse/JDK-8037096
> The fix is available at: 
> http://cr.openjdk.java.net/~pchelko/9/8037096/webrev/ 
> <http://cr.openjdk.java.net/%7Epchelko/9/8037096/webrev/>
>
> We've used a pattern with passing the parameters using an 
> MutableArray, but this array leaked.
> Now it doesn't.
>
> With best regards. Petr.
>
>


-- 
Best regards, Sergey.

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/awt-dev/attachments/20140331/542c1d16/attachment-0001.html>


More information about the awt-dev mailing list