[OpenJDK 2D-Dev] RFR: JDK-8211693 Convert C-style array declarations in client demos and jdk.accessibility

Sergey Bylokhov Sergey.Bylokhov at oracle.com
Sat Oct 6 05:34:20 UTC 2018


Looks fine.
I there are no objections I'll push the fix on Monday.

On 03/10/2018 20:46, Tagir Valeev wrote:
> Please review and sponsor the conversion of C-style arrays in other
> client modules: demos and jdk.accessibility (a left-over after
> JDK-8211300):
> http://cr.openjdk.java.net/~tvaleev/webrev/8211693/r1/
> https://bugs.openjdk.java.net/browse/JDK-8211693
> 
> This patch is much smaller, only 75 files are affected (71 in demos
> and 4 in accessibility). The procedure was the same as previously. I
> manually fixed these two compound declarations:
> 
> 1. Font2DTest.java:
> -            String fileText, textLines[];
> +            String fileText;
> +            String[] textLines;
> 
> 2. java2d/Intro.java
> -            private Shape shapes[], txShapes[];
> +            private Shape[] shapes, txShapes;
> 
> Everything else was nicely converted automatically. This time I
> reviewed the whole patch by eyes and haven't noticed any formatting
> issues.
> 
> Other two client modules java.datatransfer and jdk.unsupported.desktop
> were also checked and no C-style array declarations were found there.
> 
> With best regards,
> Tagir Valeev.
> 


-- 
Best regards, Sergey.


More information about the 2d-dev mailing list