<html>
  <head>
    <meta content="text/html; charset=UTF-8" http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    Looks mostly OK.<br>
    <br>
    I am wondering why you took out the unordered list here :<br>
<a class="moz-txt-link-freetext" href="https://docs.oracle.com/javase/9/docs/api/java/awt/color/ICC_ProfileRGB.html">https://docs.oracle.com/javase/9/docs/api/java/awt/color/ICC_ProfileRGB.html</a><br>
    <br>
    The specdiff *maybe* doesn't really tell me what it looks like now
    but I don't see how<br>
    it can still be a list ...<br>
    <br>
    May I assume you tested out any newly added links ?<br>
    eg<br>
    <meta http-equiv="content-type" content="text/html; charset=UTF-8">
    <pre><span class="new">  73  * spaces via {@link ColorSpace#getInstance}.</span></pre>
    Looks straightforward so I don't expect any issues, just checking.<br>
    <br>
    Also it looks as if some lines got longer due to reformatting, are
    they are still within bounds ?<br>
    <br>
    eg<br>
    <meta http-equiv="content-type" content="text/html; charset=UTF-8">
    <pre><span class="removed">- *</span>
<span class="removed">- * A subclass of the ICC_Profile class which represents profiles</span>
<span class="removed">- * which meet the following criteria: the color space type of the</span>
<span class="removed">- * profile is TYPE_GRAY and the profile includes the grayTRCTag and</span>
<span class="removed">- * mediaWhitePointTag tags.  Examples of this kind of profile are</span>
<span class="removed">- * monochrome input profiles, monochrome display profiles, and</span>
<span class="removed">- * monochrome output profiles.  The getInstance methods in the</span>
<span class="removed">- * ICC_Profile class will</span>
<span class="removed">- * return an ICC_ProfileGray object when the above conditions are</span>
<span class="removed">- * met.  The advantage of this class is that it provides a lookup</span>
<span class="removed">- * table that Java or native methods may be able to use directly to</span>
<span class="removed">- * optimize color conversion in some cases.</span>
<span class="new">+ * The {@code ICC_ProfileGray} class is a subclass of the {@code ICC_Profile}</span>
<span class="new">+ * class that represents profiles which meet the following criteria: the color</span>
<span class="new">+ * space type of the profile is {@code TYPE_GRAY} and the profile includes the</span>
<span class="new">+ * {@code grayTRCTag} and {@code mediaWhitePointTag} tags. The</span>
<span class="new">+ * {@code getInstance} methods in the {@code ICC_Profile} class will return an</span>
<span class="new">+ * {@code ICC_ProfileGray} object when the above conditions are met. Examples of</span>
<span class="new">+ * this kind of profile are monochrome input profiles, monochrome display</span>
<span class="new">+ * profiles, and monochrome output profiles.</span>
<span class="new">+ * <p></span>
<span class="new">+ * The advantage of this class is that it provides a lookup table that Java</span>
<span class="new">+ * or native methods can use directly to optimize color conversion in some</span>
<span class="new">+ * cases.

</span></pre>
    <br>
    -phil<br>
    <br>
    On 10/24/18, 9:36 PM, Sergey Bylokhov wrote:
    <blockquote
      cite="mid:8ccf2fc9-6050-f040-1c07-ddc61c12ab9a@oracle.com"
      type="cite">Hi, Krishna.
      <br>
      Thank you for review!
      <br>
      <br>
      <blockquote type="cite">1894, 1895, 1902, 1904.
        <br>
      </blockquote>
      The text on the lines above is a block comment not a javadoc, the
      tags are not necessary there.
      <br>
      <br>
      <blockquote type="cite">1570, 1601, </blockquote>
      <br>
      I have fixed these:
      <br>
      <a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~serb/8212790/webrev.01">http://cr.openjdk.java.net/~serb/8212790/webrev.01</a>
      <br>
      <br>
    </blockquote>
  </body>
</html>