RFR: JDK-8285676: Add missing @param tags for type parameters on classes and interfaces
Daniel Fuchs
dfuchs at openjdk.java.net
Wed Apr 27 10:59:43 UTC 2022
On Tue, 26 Apr 2022 22:24:26 GMT, Joe Darcy <darcy at openjdk.org> wrote:
> To enable more complete doclint checking (courtesy @jonathan-gibbons), please review this PR to add type-level @param tags where they are missing.
>
> To the maintainers of java.util.concurrent, those changes could be separated out in another bug if that would ease maintenance of that code.
>
> Making these library fixes is a blocker for correcting and expanding the doclint checks (JDK-8285496).
>
> I'll update copyright years before pushing.
Hi Joe, just two suggestions about the javax.management changes. Otherwise looks good!
src/java.management/share/classes/javax/management/openmbean/ArrayType.java line 96:
> 94: *
> 95: * @param <T> the Java type that instances described by this type must
> 96: * have.
Instead of "instances" - would it be more correct to say "array elements"?
src/java.management/share/classes/javax/management/openmbean/SimpleType.java line 60:
> 58: * @param <T> the Java type that instances described by this type must
> 59: * have.
> 60: *
I would suggest to say "that values described by this type"...
-------------
PR: https://git.openjdk.java.net/jdk/pull/8410
More information about the security-dev
mailing list