<AWT Dev> [9] Review Request: 6622944 Use package-info.java instead of package.html within awt packages

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

<AWT Dev> [9] Review Request: 6622944 Use package-info.java instead of package.html within awt packages

Sergey Bylokhov
Hello,
Please review the fix for jdk9.
 - All "package.html" files were replaced by "package-info.java" in java.awt package and its subpackages.
 - All <tag> were replaced by {@tag }.
 - Some unnecessary <B> tags were removed.
 - I also wraped some references to the classes in the {link}
 - All files were manually reformatted - 80 chars per line.

Note that the changeset which will be pushed, the "html" files will be marked as moved and updated, instead of removed and added.
Later I will do the same for javax.swing, and then for all other packages in java client.

Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: <AWT Dev> [9] Review Request: 6622944 Use package-info.java instead of package.html within awt packages

joe darcy
Hi Sergey,

The changes look fine; thanks,

-Joe


On 2/15/2017 6:22 AM, Sergey Bylokhov wrote:

> Hello,
> Please review the fix for jdk9.
>  - All "package.html" files were replaced by "package-info.java" in
> java.awt package and its subpackages.
>  - All <tag> were replaced by {@tag }.
>  - Some unnecessary <B> tags were removed.
>  - I also wraped some references to the classes in the {link}
>  - All files were manually reformatted - 80 chars per line.
>
> Note that the changeset which will be pushed, the "html" files will be
> marked as moved and updated, instead of removed and added.
> Later I will do the same for javax.swing, and then for all other
> packages in java client.
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-6622944
> Webrev can be found at:
> http://cr.openjdk.java.net/~serb/6622944/webrev.00/ 
> <http://cr.openjdk.java.net/%7Eserb/6622944/webrev.00/>
> Specdiff:
> http://cr.openjdk.java.net/~serb/6622944/specdiff/overview-summary.html 
> <http://cr.openjdk.java.net/%7Eserb/6622944/specdiff/overview-summary.html>

Loading...