On Fri, 20 Mar 2026 11:35:35 GMT, Marius Hanl <[email protected]> wrote:

> This PR fixes the bug where list changes could be wrong (incorrect index). 
> Nearly everything was already in place, but one usecase that had a minor flaw.
> 
> Fix is simple, the index we use for iterating must be adjusted in case we 
> remove a change from `addRemoveChanges`.
> 
> All tests are green and I also added the Tests that were in the ticket. 
> Adjusted to JUnit 5 and with a bit of explaination which helped me a lot to 
> better understand the issue and fix it.

This pull request has now been integrated.

Changeset: 6315e1f7
Author:    Marius Hanl <[email protected]>
URL:       
https://git.openjdk.org/jfx/commit/6315e1f75d3939b26d08cbcfbc496ced386def8f
Stats:     188 lines in 2 files changed: 184 ins; 0 del; 4 mod

8208758: ListChangeBuilder doesn't shift indices after idempotent change pairs

Reviewed-by: angorya, mstrauss

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

PR: https://git.openjdk.org/jfx/pull/2117

Reply via email to