Looks mostly OK.

I am wondering why you took out the unordered list here :
https://docs.oracle.com/javase/9/docs/api/java/awt/color/ICC_ProfileRGB.html

The specdiff *maybe* doesn't really tell me what it looks like now but I don't see how
it can still be a list ...

May I assume you tested out any newly added links ?
eg

  73  * spaces via {@link ColorSpace#getInstance}.

Looks straightforward so I don't expect any issues, just checking.

Also it looks as if some lines got longer due to reformatting, are they are still within bounds ?

eg

- *
- * A subclass of the ICC_Profile class which represents profiles
- * which meet the following criteria: the color space type of the
- * profile is TYPE_GRAY and the profile includes the grayTRCTag and
- * mediaWhitePointTag tags.  Examples of this kind of profile are
- * monochrome input profiles, monochrome display profiles, and
- * monochrome output profiles.  The getInstance methods in the
- * ICC_Profile class will
- * return an ICC_ProfileGray object when the above conditions are
- * met.  The advantage of this class is that it provides a lookup
- * table that Java or native methods may be able to use directly to
- * optimize color conversion in some cases.
+ * The {@code ICC_ProfileGray} class is a subclass of the {@code ICC_Profile}
+ * class that represents profiles which meet the following criteria: the color
+ * space type of the profile is {@code TYPE_GRAY} and the profile includes the
+ * {@code grayTRCTag} and {@code mediaWhitePointTag} tags. The
+ * {@code getInstance} methods in the {@code ICC_Profile} class will return an
+ * {@code ICC_ProfileGray} object when the above conditions are met. Examples 
of
+ * this kind of profile are monochrome input profiles, monochrome display
+ * profiles, and monochrome output profiles.
+ *<p>
+ * The advantage of this class is that it provides a lookup table that Java
+ * or native methods can use directly to optimize color conversion in some
+ * cases.



-phil

On 10/24/18, 9:36 PM, Sergey Bylokhov wrote:
Hi, Krishna.
Thank you for review!

1894, 1895, 1902, 1904.
The text on the lines above is a block comment not a javadoc, the tags are not necessary there.

1570, 1601,

I have fixed these:
http://cr.openjdk.java.net/~serb/8212790/webrev.01

Reply via email to