Re: RFR: 8349758: Memory leak in TreeTableView

2025-02-18 Thread Andy Goryachev
On Mon, 17 Feb 2025 20:38:30 GMT, Marius Hanl wrote: > Looks good to me too, but a test would be nice as well Similarly to https://github.com/openjdk/jfx/pull/1705#issuecomment-2652002160 , the `NodeInitializationStressTest` _is_ the test. - PR Comment: https://git.openjdk.org/jfx

Re: RFR: 8349758: Memory leak in TreeTableView

2025-02-17 Thread Marius Hanl
On Tue, 11 Feb 2025 21:16:32 GMT, Andy Goryachev wrote: > ## Root Cause > > An event handler was installed on the root property without removing it when > the root changes. > > ## Solution > > Replaced the weak parts with a change listener to the root property to ensure > correct handling of

Re: RFR: 8349758: Memory leak in TreeTableView

2025-02-16 Thread Ambarish Rapte
On Tue, 11 Feb 2025 21:16:32 GMT, Andy Goryachev wrote: > ## Root Cause > > An event handler was installed on the root property without removing it when > the root changes. > > ## Solution > > Replaced the weak parts with a change listener to the root property to ensure > correct handling of

Re: RFR: 8349758: Memory leak in TreeTableView

2025-02-14 Thread Andy Goryachev
On Tue, 11 Feb 2025 21:16:32 GMT, Andy Goryachev wrote: > ## Root Cause > > An event handler was installed on the root property without removing it when > the root changes. > > ## Solution > > Replaced the weak parts with a change listener to the root property to ensure > correct handling of

RFR: 8349758: Memory leak in TreeTableView

2025-02-11 Thread Andy Goryachev
## Root Cause An event handler was installed on the root property without removing it when the root changes. ## Solution Replaced the weak parts with a change listener to the root property to ensure correct handling of the root value changes. - Commit messages: - cleanup - remo