<html>
  <head>
    <meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
  </head>
  <body text="#000000" bgcolor="#FFFFFF">
    looks good,<br>
    Vicente<br>
    <br>
    <div class="moz-cite-prefix">On 1/8/19 1:43 AM, Joe Darcy wrote:<br>
    </div>
    <blockquote type="cite"
      cite="mid:1d4a87a8-7664-9062-7662-6e5e773f7982@oracle.com">
      <meta http-equiv="content-type" content="text/html; charset=UTF-8">
      <p>Hello,</p>
      <p>A pass over the javax.annotation.processing and
        javax.lang.model specs revealed various areas for improvement.</p>
      <p>Please review the fixes to address those issues in the webrev
        and the corresponding patch below:</p>
      <p>    <a class="moz-txt-link-freetext"
          href="http://cr.openjdk.java.net/~darcy/8216335.0/"
          moz-do-not-send="true">http://cr.openjdk.java.net/~darcy/8216335.0/</a><br>
      </p>
      <p>Thanks,<br>
      </p>
      <p>-Joe</p>
      <pre style="color: rgb(0, 0, 0); font-style: normal; font-variant-ligatures: normal; font-variant-caps: normal; font-weight: 400; letter-spacing: normal; orphans: 2; text-align: start; text-indent: 0px; text-transform: none; widows: 2; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration-style: initial; text-decoration-color: initial; overflow-wrap: break-word; white-space: pre-wrap;">--- 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
  * &quot;path-rootless&quot; rule of <a
- * href=<a class="moz-txt-link-rfc2396E" href="http://www.ietf.org/rfc/rfc3986.txt" moz-do-not-send="true">"http://www.ietf.org/rfc/rfc3986.txt"</a>>RFC 3986</a>, section
+ * href=<a class="moz-txt-link-rfc2396E" href="http://www.ietf.org/html/rfc3986.txt" moz-do-not-send="true">"http://www.ietf.org/html/rfc3986.txt"</a>>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);
 </pre>
    </blockquote>
    <br>
  </body>
</html>