[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openjdk-openjfx-dev
Subject:    Integrated: 8262518: SwingNode.setContent does not close previous content, resulting in memory leak
From:       Prasanta Sadhukhan <psadhukhan () openjdk ! org>
Date:       2023-08-29 3:51:20
Message-ID: wEH4J9pEgx-ZJzC1VB3c8TvPwk1hbwIZRFHrvNQo-3g=.4618e736-6193-4374-8754-493f944bae86 () github ! com
[Download RAW message or body]

On Tue, 22 Aug 2023 09:54:11 GMT, Prasanta Sadhukhan <psadhukhan@openjdk.org> wrote:

> Issue is when setting the content of a SwingNode, the old content is not garbage \
> collected owing to the fact JLightweightFrame is never being released by \
> SwingNodeDisposer 
> The SwingNodeDisposer holds an hard pointer to the JLightweightFrame that prevents \
> its collection 
> Modified `SwingNode.setContentImpl`  function to use a WeakReference to properly \
> release the memory.

This pull request has now been integrated.

Changeset: beca88c1
Author:    Prasanta Sadhukhan <psadhukhan@openjdk.org>
URL:       https://git.openjdk.org/jfx/commit/beca88c16fa0942ce7ed32530c411ed9671ad925
                
Stats:     19 lines in 2 files changed: 17 ins; 0 del; 2 mod

8262518: SwingNode.setContent does not close previous content, resulting in memory \
leak

Reviewed-by: angorya, jhendrikx

-------------

PR: https://git.openjdk.org/jfx/pull/1219


[prev in list] [next in list] [prev in thread] [next in thread]