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

Tue May 9 15:40:32 UTC 2023

On Mon, 8 May 2023 19:28:54 GMT, Andy Goryachev <angorya at openjdk.org> wrote:

> I agree that the idea of explicitly clearing the cells when they are not needed would be a much better solution. It will, however, require non-trivial changes in VirtualFlow, would impact three controls (List|Tree|TableView) and would require some careful testing.

Yes, and we wouldn't do this as part of a regression bug fix like this. It seems worth filing a follow-up bug for this. An explicit life-cycle that eliminates the need for weak listeners is generally a good thing.

> For the purposes of fixing the regression bug, I would like to proceed with this fix as is.

Agreed.

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

PR Comment: https://git.openjdk.org/jfx/pull/1129#issuecomment-1540416777