RFR: 8350048: Enforce threading restrictions for show and hide methods in Window, Control, and Skin [v5]
Andy Goryachev
angorya at openjdk.org
Fri Mar 7 15:33:00 UTC 2025
On Fri, 7 Mar 2025 13:54:45 GMT, Michael Strauß <mstrauss at openjdk.org> wrote:
>> Andy Goryachev has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains seven commits:
>>
>> - Merge remote-tracking branch 'origin/master' into 8350048.enforce
>> - review comments
>> - review comments
>> - Merge remote-tracking branch 'origin/master' into 8350048.enforce
>> - fixed node init test
>> - all tests
>> - initial test
>
> modules/javafx.graphics/src/main/java/javafx/stage/Stage.java line 1187:
>
>> 1185: * This call is equivalent to {@code hide()}.
>> 1186: * @throws IllegalStateException if this method is called on a thread
>> 1187: * other than the JavaFX Application Thread.
>
> Minor: add a blank line before `@throws`. I also think it's easier to read docs when multi-line text for a javadoc tag is indented, either by four spaces or lined up with the beginning of the first line of text (i.e. with the beginning of `IllegalStateException`).
good idea, even though it does not affect generated javadoc.
will do next time, to avoid re-setting the approvals ;-)
-------------
PR Review Comment: https://git.openjdk.org/jfx/pull/1717#discussion_r1985268281
More information about the openjfx-dev
mailing list