On Mon, 12 Apr 2021 13:59:13 GMT, Nir Lisker <nlis...@openjdk.org> wrote:

>> Jeanette Winzenburg has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   replaced param name in changeListener methods
>
> modules/javafx.controls/src/main/java/javafx/scene/control/SkinBase.java line 
> 286:
> 
>> 284:      * Registers an operation to perform when the given {@code 
>> observableList} sends a list change event.
>> 285:      * Does nothing if either {@code observableList} or {@code 
>> operation} are {@code null}.
>> 286:      * If multiple operations are registered on the same 
>> observableList, they will be performed in the
> 
> For "observableList" I would use "observable list" similar to what was done 
> with the other methods.

changed description as suggested, thanks :)

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

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

Reply via email to