RFR: 8262518: SwingNode.setContent does not close previous content, resulting in memory leak [v3]

Prasanta Sadhukhan psadhukhan at openjdk.org
Wed Aug 23 13:21:00 UTC 2023


> 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.

Prasanta Sadhukhan has updated the pull request incrementally with one additional commit since the last revision:

  DisposerRecord leak fix

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

Changes:
  - all: https://git.openjdk.org/jfx/pull/1219/files
  - new: https://git.openjdk.org/jfx/pull/1219/files/66b4d791..a388e83a

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jfx&pr=1219&range=02
 - incr: https://webrevs.openjdk.org/?repo=jfx&pr=1219&range=01-02

  Stats: 5 lines in 2 files changed: 1 ins; 0 del; 4 mod
  Patch: https://git.openjdk.org/jfx/pull/1219.diff
  Fetch: git fetch https://git.openjdk.org/jfx.git pull/1219/head:pull/1219

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


More information about the openjfx-dev mailing list