On Thu, 3 Oct 2024 19:31:06 GMT, Attila Szegedi <att...@openjdk.org> wrote:

>> Fixes a regression with #17818 where `ArrayList.subList(…).sort()` started 
>> incrementing `ArrayList.modCount` resulting in some cases throwing a 
>> `ConcurrentModificationException` where none was thrown before.
>> 
>> This change keeps the optimization from #17818 but restores the behavior 
>> where only sorting the `ArrayList` changes the mod count, but sorting its 
>> sublists does not.
>
> Attila Szegedi has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   - Rename test to more descriptive
>   - Add a test case for whole list sorting

Thanks for the updates! I've run these changes through our internal testing 
system and the results look good.

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

Marked as reviewed by smarks (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/21250#pullrequestreview-2348901655

Reply via email to