On Fri, 21 Jun 2024 12:11:21 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: > > Add forRemoval Now that I looked at the code more thoroughly, the no-arg constructor makes more sense actually. https://github.com/openjdk/jdk/blob/c41293a70834a79c79e859ebcdb8869884ac87dc/src/java.desktop/share/classes/javax/swing/plaf/basic/BasicSliderUI.java#L202-L207 I was under impression that `BasicSliderUI(JSlider b)` did use its parameter but it doesn't. Should we keep the new constructor and deprecate the *old* one? Should we remove the new constructor and keep the things as they've always been? ------------- PR Comment: https://git.openjdk.org/jdk/pull/19819#issuecomment-2182916810
