Request for review: 7124524 - OutOfMemory exception after (or even before) some 2500 creations of LWWindowPeer (original) (raw)
Anthony Petrov anthony.petrov at oracle.com
Tue Jan 17 07:59:36 PST 2012
- Previous message: Request for review: 7124524 - OutOfMemory exception after (or even before) some 2500 creations of LWWindowPeer
- Next message: Request for review: 7124524 - OutOfMemory exception after (or even before) some 2500 creations of LWWindowPeer
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
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/
- Previous message: Request for review: 7124524 - OutOfMemory exception after (or even before) some 2500 creations of LWWindowPeer
- Next message: Request for review: 7124524 - OutOfMemory exception after (or even before) some 2500 creations of LWWindowPeer
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]