<Swing Dev> [9] Review Request: 8175266 Use package-info.java instead of package.html within swing packages

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

<Swing Dev> [9] Review Request: 8175266 Use package-info.java instead of package.html within swing packages

Sergey Bylokhov
Hello,
Please review the fix for jdk9.
 - All "package.html" files were replaced by "package-info.java" in javax.swing 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.

Later I will do the same for all other packages in java client.


The same bug was fixed in awn already:
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: <Swing Dev> [9] Review Request: 8175266 Use package-info.java instead of package.html within swing packages

Alexandr Scherbatiy

The fix looks good to me.

Thanks,
Alexandr.

On 2/20/2017 5:05 PM, Sergey Bylokhov wrote:

> Hello,
> Please review the fix for jdk9.
>  - All "package.html" files were replaced by "package-info.java" in
> javax.swing 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.
>
> Later I will do the same for all other packages in java client.
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8175266
> Webrev can be found at:
> http://cr.openjdk.java.net/~serb/8175266/webrev.00/ 
> <http://cr.openjdk.java.net/%7Eserb/8175266/webrev.00/>
> Specdiff:
> http://cr.openjdk.java.net/~serb/8175266/specdiff/overview-summary.html 
> <http://cr.openjdk.java.net/%7Eserb/8175266/specdiff/overview-summary.html>
>
> The same bug was fixed in awn already:
> https://bugs.openjdk.java.net/browse/JDK-6622944
> http://mail.openjdk.java.net/pipermail/awt-dev/2017-February/012610.html

Loading...