On Fri, 21 Jun 2024 18:33:37 GMT, Alexey Ivanov <aiva...@openjdk.org> wrote:
>> Prasanta Sadhukhan has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Add forRemoval
>
> src/java.desktop/share/classes/javax/swing/plaf/basic/BasicSliderUI.java line
> 153:
>
>> 151: /**
>> 152: * Constructs a {@code BasicSliderUI}.
>> 153: * @deprecated This constructor will be removed in future release
>
> Sounds reasonable enough. It's what happened, isn't it? It's the reason why
> we're deprecating it and planning to remove it.
>
> Suggestion:
>
> * @deprecated This constructor was added by accident. Do not use it.
> * This constructor will be removed in a future release.
>
>
> There are comments that say this method shouldn't have been public.
>
> On the other hand, there are quite a few methods and classes in the list of
> terminally terminated elements which don't say anything at all.
Or:
* @deprecated This constructor will be removed in a future release.
* Use {@link #BasicSliderUI(JSlider b)} instead.
This is in the gist of the deprecation message for
[`SecurityManager`](https://download.java.net/java/early_access/jdk24/docs/api/java.base/java/lang/SecurityManager.html).
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/19819#discussion_r1649319001
