On Fri, 27 Sep 2024 09:18:11 GMT, Pavel Rappo <pra...@openjdk.org> wrote:

>> The is the initial version of a PR to defined an anchor point and index term 
>> of "wrapper classes."
>
> src/java.base/share/classes/java/lang/package-info.java line 36:
> 
>> 34:  * The <dfn>{@index "wrapper classes"}</dfn> {@link Boolean},
>> 35:  * {@link Character}, {@link Short}, {@link Integer}, {@link Long}, 
>> {@link Float},
>> 36:  * and {@link Double} serve this purpose.  An object of type {@code
> 
> Somehow, `Byte` is missing from that list. To be fair, it had been missing 
> before this PR.

Good catch. After Byte is added, all 8 wrapper classes are present and 
accounted for.

> src/java.base/share/classes/java/lang/package-info.java line 63:
> 
>> 61:  * security policies.
>> 62:  *
>> 63:  * <p>Class {@link Throwable} encompasses objects that may be thrown
> 
> We already have a link to `Class` in this doc comment. This one can stay 
> `{@code Class}`, which might be easier to read.

Isn't that the first reference to "Throwable" and the only link?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/21215#discussion_r1779326113
PR Review Comment: https://git.openjdk.org/jdk/pull/21215#discussion_r1779326500

Reply via email to