RFR: 8303740 JavaFX - Leak in Logging, Logging remembers last exception [v2]
Kevin Rushforth
kcr at openjdk.org
Fri Apr 7 22:38:51 UTC 2023
On Thu, 23 Mar 2023 08:31:41 GMT, Florian Kirmaier <fkirmaier at openjdk.org> wrote:
>> When an exception is logged in JavaFX, then the exception is kept in a reference.
>> This way, always the last logged exception is retained.
>>
>> This is a memory-leak.
>> This was done to write unit-tests to ensure certain error-cases are logged.
>>
>> A simple fix is, to add a flag, to enable/disable retaining the exception.
>
> Florian Kirmaier has updated the pull request incrementally with one additional commit since the last revision:
>
> JDK-8303740
> Added changes based on code review
Looks good.
-------------
Marked as reviewed by kcr (Lead).
PR Review: https://git.openjdk.org/jfx/pull/1053#pullrequestreview-1376639795
More information about the openjfx-dev
mailing list