On Thu, 15 Apr 2021 12:55:56 GMT, Kevin Rushforth <[email protected]> wrote:
>> modules/javafx.controls/src/main/java/com/sun/javafx/scene/control/LambdaMultiplePropertyChangeListenerHandler.java
>> line 63:
>>
>>> 61: */
>>> 62: public final class LambdaMultiplePropertyChangeListenerHandler {
>>> 63: // FIXME: name doesn't fit after widening to support more notification
>>> event types
>>
>> Maybe `MultipleObservableListenersHandler`? I didn't look at the class too
>> much.
>
> I like `MultipleObservableListenersHandler` or
> `LambdaMultipleObservableListenersHandler` if you want to keep `Lambda` in
> the name (not sure it matters much).
sorry, somehow this comment didn't reach my consciousness ;)
`MultipleObservableListenersHandler` sounds fine. Would like to extract the
rename to a follow-up issue, though, because it's not only used by SkinBase.
-------------
PR: https://git.openjdk.java.net/jfx/pull/409
