[9] Review Request: 8176046 Replace package.html files with package-info.java in the java.desktop module

classic Classic list List threaded Threaded
2 messages Options
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[9] Review Request: 8176046 Replace package.html files with package-info.java in the java.desktop module

Sergey Bylokhov
Hello,
Please review the fix for jdk9. This is the last request which covers all packages in java.desktop(except javax.swing and java.awt which were covered earlier)
 - All "package.html" files were replaced by "package-info.java", even for internal packages like eawt.
 - All <tag> were replaced by {@tag }.
 - All files were manually reformatted - 80 chars per line.
 - <sup><font size="-2">TM</font></sup> was replaced by &trade; This is why the specdiff contains some diffs.

Bug: https://bugs.openjdk.java.net/browse/JDK-8176046
Webrev can be found at: http://cr.openjdk.java.net/~serb/8176046/webrev.00/
Specdiff: http://cr.openjdk.java.net/~serb/8176046/specdiff/overview-summary.html
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: <Swing Dev> [9] Review Request: 8176046 Replace package.html files with package-info.java in the java.desktop module

Alexandr Scherbatiy
The fix looks good to me.

Thanks,
Alexandr.

On 3/1/2017 9:37 PM, Sergey Bylokhov wrote:

> Hello,
> Please review the fix for jdk9. This is the last request which covers all packages in java.desktop(except javax.swing and java.awt which were covered earlier)
>   - All "package.html" files were replaced by "package-info.java", even for internal packages like eawt.
>   - All <tag> were replaced by {@tag }.
>   - All files were manually reformatted - 80 chars per line.
>   - <sup><font size="-2">TM</font></sup> was replaced by &trade; This is why the specdiff contains some diffs.
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8176046
> Webrev can be found at: http://cr.openjdk.java.net/~serb/8176046/webrev.00/
> Specdiff: http://cr.openjdk.java.net/~serb/8176046/specdiff/overview-summary.html

Loading...