Request for review: 7124524 - OutOfMemory exception after (or even before) some 2500 creations of LWWindowPeer
Anthony Petrov
anthony.petrov at oracle.com
Tue Jan 17 07:59:36 PST 2012
Hi Sergey,
The fix looks good. Just one question: in LWWindowPeer.destroyBuffers()
you're removing a call to replaceSurfaceData(). Are you sure that this
is OK?
--
best regards,
Anthony
On 1/17/2012 5:02 PM, Sergey Bylokhov wrote:
> Hi Everyone,
> This is a fix for 4 memory leaks.
> 1. LWWindowPeer does not destroy backbuffer in disposeImpl().
> 2. LWToolkit stores unused links to Peer.
> 3. Local references were not deleted in the AWTWindow.m, but according
> JNFJObjectWrapper.jObjectWithEnv documentation "returns a new local-ref,
> must be released with DeleteLocalRef".
> 4. OGLContext in some cases can cache CGLSurfaceData in this case our
> LWWindowPeer was not collected.
>
> Bug: http://monaco.us.oracle.com/detail.jsf?cr=7124524
> Webrev can be found at: http://cr.openjdk.java.net/~serb/7124524/webrev.00/
>
More information about the macosx-port-dev
mailing list