<div dir="ltr"><div>Hi,</div><div><br></div><div>I've signed the OCA already, it was submitted on 03-24-2023, it's awaiting approval.</div><div><br></div><div>Cheers,</div><div>Al<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Mar 27, 2023 at 4:57 PM Pavel Rappo <<a href="mailto:pavel.rappo@oracle.com">pavel.rappo@oracle.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">To consider your patch we need you to sign OCA; have a look here: <a href="https://openjdk.org/guide/#contributing-to-an-openjdk-project" rel="noreferrer" target="_blank">https://openjdk.org/guide/#contributing-to-an-openjdk-project</a><br>
<br>
> On 24 Mar 2023, at 18:59, Aaloan Miftah <<a href="mailto:al@poly-gamma.org" target="_blank">al@poly-gamma.org</a>> wrote:<br>
> <br>
> Hello,<br>
> <br>
> Currently when a primitive type is annotated with a `@Documented` annotation, the fqn of the annotation(s)<br>
> is always included in the resulting javadoc, and links aren't properly inserted. For example,<br>
> <br>
> ``<br>
> public abstract void foo(@org.checkerframework.checker.signedness.qual.Unsigned int bar);<br>
> ``<br>
> <br>
> The `bar` parameter, in the resulting javadoc, is documented with the fqn<br>
> `org.checkerframework.checker.signedness.qual.Unsigned` annotation, instead of just `Unsigned` with a link<br>
> to the annotation.<br>
> <br>
> This patch fixes jdk.javadoc.internal.doclets.formats.html.HtmlLinkFactory::getLink() by introducing special<br>
> handling for primitive types that takes their use annotation into account, and prints the simple type name.<br>
> Note that the type name is currently being derived using PrimitiveType::getKind() instead of<br>
> utils::getTypeName(), this is because the toString() for the default case in getTypeName() will always include<br>
> the annotations, which we want to avoid. I didn't modify utils::getTypeName() since I wasn't sure if a primitive<br>
> specific visit method would mess with any expected behavior.<br>
> <br>
> Cheers,<br>
> Al<br>
> <patch.diff><br>
<br>
</blockquote></div>