Hi, Petr.
The fix looks good.

On 5/8/14 4:29 PM, Petr Pchelko wrote:
Hello,

Is anybody willing to make a second review on this fix?

With best regards. Petr.

On 30.04.2014, at 13:42, Alexander Zvegintsev <[email protected]> 
wrote:

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 <[email protected]> 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.


--
Best regards, Sergey.

Reply via email to