RFR: 8295500: AccordionSkin: memory leak when changing skin [v2]
Andy Goryachev
angorya at openjdk.org
Thu Dec 1 17:09:53 UTC 2022
On Wed, 30 Nov 2022 16:29:58 GMT, Andy Goryachev <angorya at openjdk.org> wrote:
>> as determined by SkinMemoryLeakTest (remove line 164) 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.ACCORDION;
>>
>>
>> caused by:
>> - adding and not removing listeners
>
> Andy Goryachev has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 27 commits:
>
> - Merge remote-tracking branch 'origin/master' into 8295500.accordion.skin
> - 8295500: cleanup
> - Merge remote-tracking branch 'origin/master' into 8295500.accordion.skin
> - Merge branch '8294809.listener.helper' into 8295500.accordion.skin
> - 8294809: generics
> - Merge branch '8294809.listener.helper' into 8295500.accordion.skin
> - 8294809: is alive
> - Merge branch '8294809.listener.helper' into 8295500.accordion.skin
> - Revert "8294809: removed weak listeners support"
>
> This reverts commit 2df4a85db638d76cacaf6c54ba669cdb3dd91a18.
> - 8295500: cleanup
> - ... and 17 more: https://git.openjdk.org/jfx/compare/0a785ae0...218c6ea9
@aghaisas : would you please review this?
-------------
PR: https://git.openjdk.org/jfx/pull/920
More information about the openjfx-dev
mailing list