On Sat, 16 May 2026 14:54:48 GMT, Marius Hanl <[email protected]> wrote:
>> I am not sure, I just kept the original semantics. Maybe there was a reason >> why it did it like this > > I think we should consider removing it. The call to `updateIndex` below will > lead to a `updateItem(int oldIndex)` call, which calls `updateItem(T item, > boolean empty)`. And this will set the item as expected. Yeah I removed it, the tests still pass ------------- PR Review Comment: https://git.openjdk.org/jfx/pull/2169#discussion_r3252972567
