On Thu, 9 Jul 2026 23:49:36 GMT, Nir Lisker <[email protected]> wrote:

>> interfaces are more expensive than abstract classes, and since it's an 
>> implementation detail and we are optimizing here, I'd vote for an abstract 
>> class.
>
> Today we can seal the interface. I doubt there's a significant cost here.

While it most likely won't have an impact on performance (the JVM will do class 
hierarchy analysis to figure out that `ListenerManager` and 
`OldValueCachingListenerManager` are the only subclasses of 
`ListenerManagerBase`), I think it's a good idea to seal this class to document 
intent.

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

PR Review Comment: https://git.openjdk.org/jfx/pull/1081#discussion_r3555563798

Reply via email to