Re: [9] Review request for 8036882 [macosx] Native memory leak in Java_sun_lwawt_macosx_CImage_nativeGetNSImageRepresentationSizes

Thu, 27 Mar 2014 12:28:07 -0700 

Hi, Alexander.
The fix looks good to me too.
Thanks.

On 3/27/14 9:37 PM, Petr Pchelko wrote:

Hello, Alexander.

The fix looks good to me.

With best regards. Petr.

27 марта 2014 г., в 7:49 после полудня, Alexander Scherbatiy 
<alexandr.scherba...@oracle.com> написал(а):


Hello,

Could you review the fix:
  bug: https://bugs.openjdk.java.net/browse/JDK-8036882
  webrev: http://cr.openjdk.java.net/~alexsch/8036882/webrev.00

autorelease is added for [[NSMutableArray alloc] init] call.

There is no a test for the fix because it is hard to write it.

Thanks,
Alexandr.




--
Best regards, Sergey.

Reply via email to