[Rev 03] RFR: 8236259: MemoryLeak in ProgressIndicator
Florian Kirmaier
fkirmaier at openjdk.java.net
Tue Mar 10 14:37:13 UTC 2020
On Thu, 5 Mar 2020 22:19:45 GMT, Kevin Rushforth <kcr at openjdk.org> wrote:
>> Florian Kirmaier has updated the pull request incrementally with one additional commit since the last revision:
>>
>> JDK-8236259
>> readded unit-test without any dependency
>
> The fix looks good to me. I've verified that it fixes the leak.
>
> The newly added test looks good as well, with some comments left inline (formatting, a missing copyright header, and a
> couple other suggestions).
@kevinrushforth
I've now cleaned it up based on your suggestions.
It would be great when Travis could validate the formatting.
It would save a lot of time for everyone.
-------------
PR: https://git.openjdk.java.net/jfx/pull/71
More information about the openjfx-dev
mailing list