RFR: 8290844: Add Skin.install() method [v2]

Andy Goryachev angorya at openjdk.org
Thu Aug 11 15:59:42 UTC 2022


On Thu, 11 Aug 2022 10:47:35 GMT, Ajit Ghaisas <aghaisas at openjdk.org> wrote:

>> Andy Goryachev has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   8290844: javadoc
>
> modules/javafx.controls/src/main/java/javafx/scene/control/Skinnable.java line 60:
> 
>> 58:      * {@link Skin#getSkinnable()} against this Skinnable,
>> 59:      * and may throw an IllegalArgumentException if it is not the same.
>> 60:      *
> 
> `@throws` tag can be used

In this case, a IllegalArgumentException is not a checked exception (not a part of general contract).

Just like in TextInputControl.getText(int, int) : 457, for example.

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

PR: https://git.openjdk.org/jfx/pull/845


More information about the openjfx-dev mailing list