On Thu, 23 Oct 2025 23:24:54 GMT, Michael Strauß <[email protected]> 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