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