RFR: 8307538: Memory leak in TreeTableView when calling refresh [v7]

Andy Goryachev angorya at openjdk.org
Mon May 15 16:12:10 UTC 2023


> Fixed a memory leak in TreeTableView by reverting to register**Listener (which is ok in this particular situation) - the leak is specific to TreeTableRowSkin.
> 
> Added a unit test.

Andy Goryachev has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains 17 additional commits since the last revision:

 - Merge remote-tracking branch 'origin/master' into 8307538.refresh
 - review comments
 - John is right
 - review comments
 - one more test case
 - back to register listener
 - weak lambdas are getting collected
 - removed monkey tester changes
 - Merge remote-tracking branch 'origin/master' into 8307538.refresh
 - whitespace
 - ... and 7 more: https://git.openjdk.org/jfx/compare/782a7150...f68de127

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

Changes:
  - all: https://git.openjdk.org/jfx/pull/1129/files
  - new: https://git.openjdk.org/jfx/pull/1129/files/e1b72a13..f68de127

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jfx&pr=1129&range=06
 - incr: https://webrevs.openjdk.org/?repo=jfx&pr=1129&range=05-06

  Stats: 1842 lines in 27 files changed: 1672 ins; 113 del; 57 mod
  Patch: https://git.openjdk.org/jfx/pull/1129.diff
  Fetch: git fetch https://git.openjdk.org/jfx.git pull/1129/head:pull/1129

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


More information about the openjfx-dev mailing list