On Sat, 11 May 2024 17:52:28 GMT, Nizar Benalla <d...@openjdk.org> wrote:

> If you're currently reviewing this PR, thank you!
> Most fixes here are according to the reports by the since checker tool in 
> #18934 and are pretty simple.
> 
> To make reviewing easier
> - `BasicSliderUI` has the constructor `public BasicSliderUI(JSlider b)` for a 
> long time so the default constructor (without parameters) didn't exist until 
> JDK 16
> 
> For the `package-info` files, it is pretty hard to find source code of JDK 
> 1-5 so I used the `grep` command to find the oldest instance of an `@since` 
> in those packages.
> 
> I found instances of `@since 1.1` in the other packages but 
> `javax/swing/plaf/synth/package-info.java` might be worth checking as most 
> classes there had no `@since`.

src/java.desktop/share/classes/javax/swing/package-info.java line 153:

> 151:  * @serial exclude
> 152:  *
> 153:  * @since 1.1

This isn't right. Where did you get this from ?
Swing only became part of the JDK in JDK 1.2
It was an unbundled library before then.
If you find any @since 1.1 tags in the Swing API they are a mistake.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/19192#discussion_r1600695814

Reply via email to