JDK 13 RFR of 8216335: Minor cleanups to javax.annotation.processing and javax.lang.model javadoc

Vicente Romero vicente.romero at oracle.com
Tue Jan 8 19:11:35 UTC 2019


looks good,
Vicente

On 1/8/19 1:43 AM, Joe Darcy wrote:
>
> Hello,
>
> A pass over the javax.annotation.processing and javax.lang.model specs 
> revealed various areas for improvement.
>
> Please review the fixes to address those issues in the webrev and the 
> corresponding patch below:
>
> http://cr.openjdk.java.net/~darcy/8216335.0/
>
> Thanks,
>
> -Joe
>
> --- old/src/java.compiler/share/classes/javax/annotation/processing/AbstractProcessor.java	2019-01-07 22:32:32.960000000 -0800
> +++ new/src/java.compiler/share/classes/javax/annotation/processing/AbstractProcessor.java	2019-01-07 22:32:32.756000000 -0800
> @@ -92,10 +92,10 @@
>        * same set of strings as the annotation.  If the class is not so
>        * annotated, an empty set is returned.
>        *
> -     * If the {@link ProcessingEnvironment#getSourceVersion source
> +     * If the {@linkplain ProcessingEnvironment#getSourceVersion source
>        * version} does not support modules, in other words if it is less
>        * than or equal to {@link SourceVersion#RELEASE_8 RELEASE_8},
> -     * then any leading {@link Processor#getSupportedAnnotationTypes
> +     * then any leading {@linkplain Processor#getSupportedAnnotationTypes
>        * module prefixes} are stripped from the names.
>        *
>        * @return the names of the annotation types supported by this
> --- old/src/java.compiler/share/classes/javax/annotation/processing/Filer.java	2019-01-07 22:32:33.376000000 -0800
> +++ new/src/java.compiler/share/classes/javax/annotation/processing/Filer.java	2019-01-07 22:32:33.180000000 -0800
> @@ -60,7 +60,7 @@
>    * by {@code '/'}; {@code '.'} and {@code '..'} are invalid path
>    * segments.  A valid relative name must match the
>    * "path-rootless" rule of <a
> - * href="http://www.ietf.org/rfc/rfc3986.txt">RFC 3986</a>, section
> + * href="http://www.ietf.org/html/rfc3986.txt">RFC 3986</a>, section
>    * 3.3.
>    *
>    * <p>The file creation methods take a variable number of arguments to
> --- old/src/java.compiler/share/classes/javax/annotation/processing/Processor.java	2019-01-07 22:32:33.940000000 -0800
> +++ new/src/java.compiler/share/classes/javax/annotation/processing/Processor.java	2019-01-07 22:32:33.748000000 -0800
> @@ -59,7 +59,7 @@
>    * constructor of the processor class.
>    *
>    * <li>Next, the tool calls the {@link #init init} method with
> - * an appropriate {@code ProcessingEnvironment}.
> + * an appropriate {@link ProcessingEnvironment}.
>    *
>    * <li>Afterwards, the tool calls {@link #getSupportedAnnotationTypes
>    * getSupportedAnnotationTypes}, {@link #getSupportedOptions
> --- old/src/java.compiler/share/classes/javax/lang/model/element/ModuleElement.java	2019-01-07 22:32:34.520000000 -0800
> +++ new/src/java.compiler/share/classes/javax/lang/model/element/ModuleElement.java	2019-01-07 22:32:34.328000000 -0800
> @@ -317,7 +317,7 @@
>   
>           /**
>            * Returns the specific modules to which the package is being exported,
> -         * or null, if the package is exported to all modules which
> +         * or {@code null}, if the package is exported to all modules which
>            * have readability to this module.
>            * @return the specific modules to which the package is being exported
>            */
> @@ -339,7 +339,7 @@
>   
>           /**
>            * Returns the specific modules to which the package is being open
> -         * or null, if the package is open all modules which
> +         * or {@code null}, if the package is open all modules which
>            * have readability to this module.
>            * @return the specific modules to which the package is being opened
>            */
> --- old/src/java.compiler/share/classes/javax/lang/model/util/Elements.java	2019-01-07 22:32:34.928000000 -0800
> +++ new/src/java.compiler/share/classes/javax/lang/model/util/Elements.java	2019-01-07 22:32:34.732000000 -0800
> @@ -511,6 +511,7 @@
>        * @param hidden  the second element
>        * @return {@code true} if and only if the first element hides
>        *          the second
> +     * @jls 8.4.8 Inheritance, Overriding, and Hiding
>        */
>       boolean hides(Element hider, Element hidden);
>   

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.java.net/pipermail/compiler-dev/attachments/20190108/9290f8f5/attachment.html>


More information about the compiler-dev mailing list