[Rev 01] RFR: 8244112: Skin implementations: must not violate contract of dispose

Ambarish Rapte arapte at openjdk.java.net
Wed May 13 10:21:47 UTC 2020


On Mon, 11 May 2020 04:51:51 GMT, Ambarish Rapte <arapte at openjdk.org> wrote:

>> Jeanette Winzenburg has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   cleanup
>>   
>>   - corrected incorrect bug id in commented
>>   - removed unrelated test change from TextAreaTest
>>   - updated copyright year
>
> modules/javafx.controls/src/main/java/javafx/scene/control/skin/MenuButtonSkinBase.java line 222:
> 
>> 221:         // FIXME : JDK-8244112 - backout if we are already disposed
>> 222:         // should check for getSkinnable to be null and return
>> 223:         if (getSkinnable() == null) return;
> 
> This comment can be removed. FIXME is getting fixed. :)

Ajit has already commented about it, please resolve this comment.

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

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


More information about the openjfx-dev mailing list