JDK 13 RFR of 8216335: Minor cleanups to javax.annotation.processing and javax.lang.model javadoc
Joseph D. Darcy
joe.darcy at oracle.com
Fri Jan 11 00:02:40 UTC 2019
Hi Jan,
Thanks for checking; that URL doesn't seem to work now for me either.
I'll file a follow-up issue to change it to
https://www.rfc-editor.org/info/rfc3986
which will hopefully be stable going forward.
Cheers,
-Joe
On 1/10/2019 7:19 AM, Jan Lahoda wrote:
> Looks OK to me except for:
> > - * href="http://www.ietf.org/rfc/rfc3986.txt">RFC 3986</a>, section
> > + * href="http://www.ietf.org/html/rfc3986.txt">RFC 3986</a>, section
>
> The new link does not seem to work for me.
>
> Jan
>
> On 8.1.2019 07:43, 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);
>>
>>
More information about the compiler-dev
mailing list