[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