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

Reply via email to