<AWT Dev> [9] Review Request: 8036917 [macosx] Native memory leaks.

[Alexander Zvegintsev]

Hello, Petr,

the fix looks good to me.

Thanks,

Alexander.

On 04/30/2014 01:04 PM, Petr Pchelko wrote:
> Hello,
>
> It's a reminder about this review.
>
> With best regards. Petr.
>
> On 21.04.2014, at 16:34, Petr Pchelko <petr.pchelko at oracle.com> wrote:
>
>> Hello, AWT Team.
>>
>> Please review the fix for the issue:
>> https://bugs.openjdk.java.net/browse/JDK-8036917
>> The fix is available at:
>> http://cr.openjdk.java.net/~pchelko/9/8036917/webrev/
>>
>> The problem:
>> We overretain native images so they are never deleted.
>>
>> The solution:
>> When we return a native image pointer to Java it's retain count should be 1,
>> so in case we use alloc-init we should not use retain. The image will be deleted
>> in CFRetainedResource.dispose
>>
>> It's impossible to make a regression test.
>>
>> With best regards. Petr.