[OpenJDK 2D-Dev] [10] Review Request: 8184435 Cleanup of javadoc in javax.print package

Sergey Bylokhov Sergey.Bylokhov at oracle.com
Sat Aug 5 01:15:22 UTC 2017


Any volunteers? =)

On 16.07.2017 16:42, Sergey Bylokhov wrote:
> Hello,
> Please review the fix for jdk10.
> The cleanup was done in the same way as for datatransfer, sound and 
> accessibility packages(see links in the CR).
> 
> I suggest to check the specdiff first, because for some methods the 
> specification was reworked. CSR will be filed after technical review.
> 
> Bug: https://bugs.openjdk.java.net/browse/JDK-8184435
> Webrev can be found at: http://cr.openjdk.java.net/~serb/8184435/webrev.07
> Specdiff: 
> http://cr.openjdk.java.net/~serb/8184435/specdiff.07/overview-summary.html
> 
> In this fix the javadoc is updated and the next rules were applied:
>   - <tag> should be replaced by {@tag }
>   - 80 column limit
>   - 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 should be removed
>   - the "null"/"true"/"false"/"this"/"ClassName" should be wrapped in 
> {@code } when necessary
>   - the order of different tags were unified across the package
> ... etc
> 
> There are also some mixing of different "reference usage", for example 
> "InputStream" vs "input stream", "String" vs "string", etc. I tried to 
> fix some of them.
> 


-- 
Best regards, Sergey.


More information about the 2d-dev mailing list