On Fri, 17 Apr 2026 22:22:05 GMT, Marius Hanl <[email protected]> wrote:

>> modules/javafx.base/src/test/java/test/javafx/collections/ListChangeBuilderTest.java
>>  line 744:
>> 
>>> 742:         list.remove("C");
>>> 743: 
>>> 744:         list.doEndChange();
>> 
>> do you think it would make sense to add scenarios with .filtered() and 
>> .sorted() described in the reproducer for 
>> https://bugs.openjdk.org/browse/JDK-8237868 as tests?
>> 
>> https://bugs.openjdk.org/secure/attachment/86526/SampleApp.java
>
> While it is not a problem inside `FilteredList` and `SortedList`, more tests 
> don't hurt. So I will have a look. Especially since we don't have many tests 
> for the `FilteredList` and `SortedList`.

Thank you so much!

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

PR Review Comment: https://git.openjdk.org/jfx/pull/2154#discussion_r3103624508

Reply via email to