RFR: 8359053: Implement JEP 504 - Remove the Applet API [v2]
Alexey Ivanov
aivanov at openjdk.org
Tue Jun 10 11:51:30 UTC 2025
On Mon, 9 Jun 2025 20:50:24 GMT, Phil Race <prr at openjdk.org> wrote:
>> src/java.desktop/share/classes/com/sun/java/swing/SwingUtilities3.java line 108:
>>
>>> 106: * Checks if vsync painting is requested for {@code rootContainer}
>>> 107: *
>>> 108: * @param rootContainer topmost container. Should be Window
>>
>> Suggestion:
>>
>> * @param rootContainer topmost container. Should be {@code Window}
>
> This isn't public API doc, so I'm not inclined to go updating it any more than I have to.
You have kept `@code` for [`setVsyncRequested`](https://github.com/openjdk/jdk/pull/25698/files/604d78c9e465abe6853f97b328a0c6f13c276a53#diff-52524c1d4891d53a8f9acc07e5e266ea7a07ee5ac3f84c41c240c26409719453R91); for consistency `isVsyncRequested` should have `{@code Window}` too.
>> src/java.desktop/share/classes/java/awt/Component.java line 3938:
>>
>>> 3936: /**
>>> 3937: * Inner class for flipping buffers on a component. That component must
>>> 3938: * be a {@code Canvas} or {@code Window}
>>
>> Suggestion:
>>
>> * be a {@code Canvas} or {@code Window}.
>>
>> It ended with a full stop, and the full stop should be preserved.
>
> added back
Still missing here.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/25698#discussion_r2137673477
PR Review Comment: https://git.openjdk.org/jdk/pull/25698#discussion_r2137677189
More information about the client-libs-dev
mailing list