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