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