RFR: 8258777: SkinBase: add api to un-/register invalidation-/listChange listeners [v2]

Kevin Rushforth kcr at openjdk.java.net
Thu Apr 15 12:58:42 UTC 2021


On Wed, 24 Mar 2021 23:51:37 GMT, Nir Lisker <nlisker at openjdk.org> wrote:

>> Jeanette Winzenburg has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   fixed missing/incorrect @since tags in new api doc
>
> 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).

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

PR: https://git.openjdk.java.net/jfx/pull/409


More information about the openjfx-dev mailing list