RFR: 8367439: Bulk change notifications for ObservableSet and ObservableMap [v3]

Andy Goryachev angorya at openjdk.org
Fri Oct 24 16:23:22 UTC 2025


On Thu, 23 Oct 2025 23:24:54 GMT, Michael Strauß <mstrauss at openjdk.org> wrote:

>> modules/javafx.base/src/main/java/com/sun/javafx/collections/ObservableMapWrapper.java line 795:
>> 
>>> 793:     }
>>> 794: 
>>> 795:     private class SimpleChange extends MapChangeListener.Change<K,V> {
>> 
>> could this and `BulkChange` classes made static?
>
> Yes, I can do that for the existing `Change` implementations, but it's not directly related to bulk change notifications. Maybe another PR would be better.

since you are already here...  saves a pointer maybe

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

PR Review Comment: https://git.openjdk.org/jfx/pull/1885#discussion_r2461190925


More information about the openjfx-dev mailing list