+1
On 6/9/2017 12:34 PM, Sergey Bylokhov wrote:
Hello, Please review the fix for jdk9. Bug: https://bugs.openjdk.java.net/browse/JDK-8181877 Webrev can be found at: http://cr.openjdk.java.net/~serb/8181877/webrev.00 Specdiff: http://cr.openjdk.java.net/~serb/8181877/specdiff.00/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 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. During the cleanup I found that some of the specs are outdated, for example "package-summary" contains incorrect number of classes/interfaces. In such cases I leaved the spec as-is with intention to rework the text later.
