Re: RFR: 8295754: PaginationSkin: memory leak when changing skin

2022-11-30 Thread Ajit Ghaisas
On Thu, 20 Oct 2022 20:48:07 GMT, Andy Goryachev wrote: > Fixes memory leaks as determined by SkinMemoryLeakTest (remove line 171) and > a leak tester > https://github.com/andy-goryachev-oracle/Test/blob/main/src/goryachev/apps/LeakTest.java > > Make sure to configure the current test in

Re: RFR: 8295754: PaginationSkin: memory leak when changing skin

2022-11-29 Thread Kevin Rushforth
On Thu, 20 Oct 2022 20:48:07 GMT, Andy Goryachev wrote: > Fixes memory leaks as determined by SkinMemoryLeakTest (remove line 171) and > a leak tester > https://github.com/andy-goryachev-oracle/Test/blob/main/src/goryachev/apps/LeakTest.java > > Make sure to configure the current test in

Re: RFR: 8295754: PaginationSkin: memory leak when changing skin

2022-11-28 Thread Andy Goryachev
On Thu, 3 Nov 2022 18:46:36 GMT, Kevin Rushforth wrote: >> modules/javafx.controls/src/main/java/javafx/scene/control/skin/PaginationSkin.java >> line 917: >> >>> 915: }); >>> 916: >>> 917: >>>

Re: RFR: 8295754: PaginationSkin: memory leak when changing skin

2022-11-28 Thread Kevin Rushforth
On Fri, 28 Oct 2022 20:08:33 GMT, Andy Goryachev wrote: >> Fixes memory leaks as determined by SkinMemoryLeakTest (remove line 171) and >> a leak tester >> https://github.com/andy-goryachev-oracle/Test/blob/main/src/goryachev/apps/LeakTest.java >> >> Make sure to configure the current test in

Re: RFR: 8295754: PaginationSkin: memory leak when changing skin

2022-11-28 Thread Andy Goryachev
On Wed, 23 Nov 2022 00:13:07 GMT, Kevin Rushforth wrote: >> Fixes memory leaks as determined by SkinMemoryLeakTest (remove line 171) and >> a leak tester >> https://github.com/andy-goryachev-oracle/Test/blob/main/src/goryachev/apps/LeakTest.java >> >> Make sure to configure the current test in

Re: RFR: 8295754: PaginationSkin: memory leak when changing skin

2022-11-28 Thread Andy Goryachev
On Thu, 20 Oct 2022 20:48:07 GMT, Andy Goryachev wrote: > Fixes memory leaks as determined by SkinMemoryLeakTest (remove line 171) and > a leak tester > https://github.com/andy-goryachev-oracle/Test/blob/main/src/goryachev/apps/LeakTest.java > > Make sure to configure the current test in

Re: RFR: 8295754: PaginationSkin: memory leak when changing skin

2022-11-28 Thread Kevin Rushforth
On Thu, 20 Oct 2022 20:48:07 GMT, Andy Goryachev wrote: > Fixes memory leaks as determined by SkinMemoryLeakTest (remove line 171) and > a leak tester > https://github.com/andy-goryachev-oracle/Test/blob/main/src/goryachev/apps/LeakTest.java > > Make sure to configure the current test in

RFR: 8295754: PaginationSkin: memory leak when changing skin

2022-11-28 Thread Andy Goryachev
Fixes memory leaks as determined by SkinMemoryLeakTest (remove line 171) and a leak tester https://github.com/andy-goryachev-oracle/Test/blob/main/src/goryachev/apps/LeakTest.java Make sure to configure the current test in LeakTest: protected final Type WE_ARE_TESTING = Type.PAGINATION; Found