On Mon, 24 Jun 2024 05:50:40 GMT, Prasanta Sadhukhan <psadhuk...@openjdk.org> 
wrote:

>> The no-arg constructor BasicSliderUI() was added under 
>> [JDK-8250852](https://bugs.openjdk.org/browse/JDK-8250852) by mistake. This 
>> constructor should be deprecated for removal in future release
>
> Prasanta Sadhukhan has updated the pull request incrementally with one 
> additional commit since the last revision:
> 
>   Added why

src/java.desktop/share/classes/javax/swing/plaf/basic/BasicSliderUI.java line 
154:

> 152:      * Constructs a {@code BasicSliderUI}.
> 153:      * @deprecated This constructor was exposed erroneously and will be 
> removed in next version.
> 154:      *             Use {@link #BasicSliderUI(JSlider b)} instead.

Did you build the docs and check that the `@link` command works? I would have 
expected it without the named parameter: `{@link #BasicSliderUI(JSlider)}`.

Also, you might want to change "next version" to "a future version" to allow 
more flexibility as to the timing of the removal.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/19819#discussion_r1650968421

Reply via email to