RFR: JDK-8236259: MemoryLeak in ProgressIndicator - DeterminateIndicator doesn't get collected
Florian Kirmaier
florian.kirmaier at gmail.com
Thu Dec 19 12:37:41 UTC 2019
Hi everyone,
Here is my PR for the memory-leak I've reported some minutes ago.
PR: https://github.com/openjdk/jfx/pull/71
Ticket: https://bugs.openjdk.java.net/browse/JDK-8236259
The fix itself is quite straight forward.
It basically just removed the listener which causes the leak.
The unit-test for the fix is a bit more complicated.
I added a library JMemoryBuddy https://github.com/Sandec/JMemoryBuddy
(written by myself), which simplifies testing for memory leaks.
I think there are many places in the JavaFX-codebase that could highly
benefit from this library.
It could also simplify many of the already existing unit tests.
It makes testing for memory-leaks readable and reliable.
It would also be possible to just copy the code of the library into the
JavaFX-codebase.
It only contains a single class.
I also had to make a method public, to write the test. I'm open to ideas,
how I could solve it differently.
Florian Kirmaier
More information about the openjfx-dev
mailing list