RFR: 8295500: AccordionSkin: memory leak when changing skin [v5]

Andy Goryachev angorya at openjdk.org
Fri Dec 2 16:48:04 UTC 2022


> 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 incrementally with one additional commit since the last revision:

  8295500: trying to avoid merge conflicts in the future

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

Changes:
  - all: https://git.openjdk.org/jfx/pull/920/files
  - new: https://git.openjdk.org/jfx/pull/920/files/d930b08f..005ada7f

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jfx&pr=920&range=04
 - incr: https://webrevs.openjdk.org/?repo=jfx&pr=920&range=03-04

  Stats: 17 lines in 1 file changed: 17 ins; 0 del; 0 mod
  Patch: https://git.openjdk.org/jfx/pull/920.diff
  Fetch: git fetch https://git.openjdk.org/jfx pull/920/head:pull/920

PR: https://git.openjdk.org/jfx/pull/920


More information about the openjfx-dev mailing list