RFR: 8184166: SortedList does not free up memory [v5]

Andy Goryachev angorya at openjdk.org
Thu Dec 18 19:41:02 UTC 2025


On Thu, 18 Dec 2025 19:38:21 GMT, Cormac Redmond <duke at openjdk.org> wrote:

>> Fix SortedList memory leak by clearing unused entries in the internal array when the source list shrinks. Also nullify tempElement.e after use,
>> 
>> Adding "--add-opens=javafx.base/javafx.collections.transformation=ALL-UNNAMED" to allow reflection access from tests.
>
> Cormac Redmond has updated the pull request incrementally with one additional commit since the last revision:
> 
>   8184166: improve readability

Thank you for making the changes, and for fixing the issue (in record time!).  This PR has all the chances to go into jfx26.

I see github is complaining about force-pushing.  We usually don't do that, merging the latest master into the PR branch instead.  Same effect, with the comments preserved.

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

Marked as reviewed by angorya (Reviewer).

PR Review: https://git.openjdk.org/jfx/pull/2000#pullrequestreview-3594737000


More information about the openjfx-dev mailing list