[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