Re: RFR: 8295809: TreeTableViewSkin: memory leak when changing skin [v4]

2022-12-08 Thread Ajit Ghaisas
On Wed, 7 Dec 2022 17:54:55 GMT, Andy Goryachev wrote: >> as determined by SkinMemoryLeakTest (remove line 180) and a leak tester >> https://github.com/andy-goryachev-oracle/Test/blob/main/src/goryachev/apps/LeakTest.java >> >> caused by: >> - adding and not removing listeners >> - adding and

Re: RFR: 8295809: TreeTableViewSkin: memory leak when changing skin [v2]

2022-12-07 Thread Andy Goryachev
On Wed, 7 Dec 2022 10:25:50 GMT, Ajit Ghaisas wrote: >> Andy Goryachev has updated the pull request with a new target base due to a >> merge or a rebase. The pull request now contains 36 commits: >> >> - 8295809: cleanup >> - Merge remote-tracking branch 'origin/master' into >>

Re: RFR: 8295809: TreeTableViewSkin: memory leak when changing skin [v4]

2022-12-07 Thread Andy Goryachev
> as determined by SkinMemoryLeakTest (remove line 180) and a leak tester > https://github.com/andy-goryachev-oracle/Test/blob/main/src/goryachev/apps/LeakTest.java > > caused by: > - adding and not removing listeners > - adding and not removing event handlers/filters > - adding and not removing

Re: RFR: 8295809: TreeTableViewSkin: memory leak when changing skin [v2]

2022-12-07 Thread Andy Goryachev
On Wed, 7 Dec 2022 10:16:36 GMT, Ajit Ghaisas wrote: >> Andy Goryachev has updated the pull request with a new target base due to a >> merge or a rebase. The pull request now contains 36 commits: >> >> - 8295809: cleanup >> - Merge remote-tracking branch 'origin/master' into >>

Re: RFR: 8295809: TreeTableViewSkin: memory leak when changing skin [v3]

2022-12-07 Thread Andy Goryachev
> as determined by SkinMemoryLeakTest (remove line 180) and a leak tester > https://github.com/andy-goryachev-oracle/Test/blob/main/src/goryachev/apps/LeakTest.java > > caused by: > - adding and not removing listeners > - adding and not removing event handlers/filters > - adding and not removing

Re: RFR: 8295809: TreeTableViewSkin: memory leak when changing skin [v2]

2022-12-07 Thread Ajit Ghaisas
On Tue, 6 Dec 2022 17:29:40 GMT, Andy Goryachev wrote: >> as determined by SkinMemoryLeakTest (remove line 180) and a leak tester >> https://github.com/andy-goryachev-oracle/Test/blob/main/src/goryachev/apps/LeakTest.java >> >> caused by: >> - adding and not removing listeners >> - adding and

Re: RFR: 8295809: TreeTableViewSkin: memory leak when changing skin [v2]

2022-12-06 Thread Andy Goryachev
> as determined by SkinMemoryLeakTest (remove line 180) and a leak tester > https://github.com/andy-goryachev-oracle/Test/blob/main/src/goryachev/apps/LeakTest.java > > caused by: > - adding and not removing listeners > - adding and not removing event handlers/filters > - adding and not removing

Re: RFR: 8295809: TreeTableViewSkin: memory leak when changing skin

2022-12-06 Thread Andy Goryachev
On Mon, 24 Oct 2022 19:06:26 GMT, Andy Goryachev wrote: > as determined by SkinMemoryLeakTest (remove line 180) and a leak tester > https://github.com/andy-goryachev-oracle/Test/blob/main/src/goryachev/apps/LeakTest.java > > caused by: > - adding and not removing listeners > - adding and not

RFR: 8295809: TreeTableViewSkin: memory leak when changing skin

2022-12-05 Thread Andy Goryachev
as determined by SkinMemoryLeakTest (remove line 180) and a leak tester https://github.com/andy-goryachev-oracle/Test/blob/main/src/goryachev/apps/LeakTest.java caused by: - adding and not removing listeners - adding and not removing event handlers/filters - adding and not removing cell factory -

Re: RFR: 8295809: TreeTableViewSkin: memory leak when changing skin

2022-12-05 Thread Andy Goryachev
On Mon, 24 Oct 2022 19:06:26 GMT, Andy Goryachev wrote: > as determined by SkinMemoryLeakTest (remove line 180) and a leak tester > https://github.com/andy-goryachev-oracle/Test/blob/main/src/goryachev/apps/LeakTest.java > > caused by: > - adding and not removing listeners > - adding and not