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

John Hendrikx jhendrikx at openjdk.org
Fri May 12 08:22:56 UTC 2023


On Thu, 11 May 2023 22:28:27 GMT, Andy Goryachev <angorya at openjdk.org> wrote:

>> Even if you are using weak listeners, `dispose` should remove them.
>
> revered back to using register*Listener, which creates a weak listener and also disconnects them in dispose()

I removed this change, and ran the tests, no failures occurred, which leads me to believe that this change is unnecessary or needs a test to cover this case.

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

PR Review Comment: https://git.openjdk.org/jfx/pull/1129#discussion_r1192058926


More information about the openjfx-dev mailing list