On Thu, 18 Dec 2025 16:02:02 GMT, Andy Goryachev <[email protected]> wrote:
>> Cormac Redmond has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> 8184166: Move nulling of out-of-range elements to inside if
>> (c.wasRemoved()) { } block, as this is only where it's required.
>
> modules/javafx.base/src/main/java/javafx/collections/transformation/SortedList.java
> line 267:
>
>> 265:
>> 266: // Null out out-of-range array elements to avoid
>> maintaining object references
>> 267: for (int i = size; i < (size + c.getRemovedSize());
>> i++) {
>
> minor suggestion:
>
> int ct = size + c.getRemovedSize();
> for (int i = size; i < ct; i++) {
Done, thanks!
-------------
PR Review Comment: https://git.openjdk.org/jfx/pull/2000#discussion_r2632386015
