[OpenJDK 2D-Dev] [12] Review Request: 8212790 Javadoc cleanup of java.awt.color package

Sergey Bylokhov Sergey.Bylokhov at oracle.com
Mon Oct 22 23:20:31 UTC 2018


Hello.
Please review a javadoc "weekend cleanup" in jdk 12.

Bug: https://bugs.openjdk.java.net/browse/JDK-8212790
Webrev: http://cr.openjdk.java.net/~serb/8212790/webrev.00
Specdiff: http://cr.openjdk.java.net/~serb/8212790/specdiff.00/overview-summary.html

To check the difference I suggest to use specdiff.

Some important changes:
  - The copyright header in CMMException.java is changed. I have checked the history and confirmed that this file had the same copyright as other files in jdk6. And I guess the header was not updated when it was moved to the open repo in JDK-6662775.
  - Descriptions of ICC_ProfileGray and ICC_ProfileRGB classes are unified. These are similar classes, but the javadoc was a little bit different. Any suggestions to improve it further are welcome.

Other changes:
  - description of the class/method/field should be followed by dot
  - @param, @return should not end with a dot, except a case when more than one sentences are used
  - empty line after description/before the first tag was added
  - unnecessary empty lines were removed
  - sets of spaces in the middle of text were deleted
  - @param, @throws, @return should be aligned, to be more readable
  - unnecessary imports were removed
  - the "true"/"false" were wrapped in {@code } when necessary
  - @exception is replaced by @throws

-- 
Best regards, Sergey.


More information about the 2d-dev mailing list