RFR: JDK-8326951 Missing @since Tags [v2]

Jaikiran Pai jpai at openjdk.org
Tue Mar 19 11:23:36 UTC 2024


On Tue, 19 Mar 2024 11:21:14 GMT, Nizar Benalla <duke at openjdk.org> wrote:

>> I added `@since` tags for methods/constructors that do not match the `@since` of the enclosing class.
>> 
>> The `write` method already existed in `PrintStream` in earlier versions and instances of it could always call this method, since it extends `FilterOutputStream` [which has the method](https://github.com/openjdk/jdk6/blob/3e49aa876353eaa215cde71eb21acc9b7f9872a0/jdk/src/share/classes/java/io/FilterOutputStream.java#L96).
>
> Nizar Benalla has updated the pull request incrementally with three additional commits since the last revision:
> 
>  - Revert "fix rest of private/public since tags"
>    
>    This reverts commit 2c04a9d8e773616b7b6239335d4e5eb955944ad1.
>  - Revert "removed unnecessary @ since tags"
>    
>    This reverts commit 5da3f0d83d19393eeb3a9da68aac40dd999de660.
>  - removed unnecessary @ since tags

src/java.base/share/classes/javax/crypto/interfaces/DHPublicKey.java line 68:

> 66:      *
> 67:      * @return {@inheritDoc java.security.AsymmetricKey}
> 68:      * @since 22

Hello Nizar, are the removal of `@since` in this PR intentional? I haven't checked all of them, but this specific removal appears incorrect, since this method was indeed introduced in Java 22 as part of https://bugs.openjdk.org/browse/JDK-8318096.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/18055#discussion_r1526987579


More information about the nio-dev mailing list