RFR: 8340078: Open source several 2D tests [v2]

Harshitha Onkar honkar at openjdk.org
Wed Sep 18 18:26:13 UTC 2024


On Wed, 18 Sep 2024 17:43:08 GMT, Phil Race <prr at openjdk.org> wrote:

>> test/jdk/sun/java2d/SunGraphics2D/RevalidateBug.java line 34:
>> 
>>> 32:  */
>>> 33: 
>>> 34: import java.awt.Color;
>> 
>> Unused import.
>
> ? I use Color multiple times.

I probably added it to the wrong test by mistake. It was for DrawRoundRect0Bug.java test which has the double Color imports - static and non-static.

>> test/jdk/sun/java2d/SunGraphics2D/ScaledPolyTest.java line 31:
>> 
>>> 29: 
>>> 30: import java.awt.Color;
>>> 31: import static java.awt.Color.*;
>> 
>> static import redudant here.
>
> fixed

Sorry somehow the comments got added to the wrong test.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/21045#discussion_r1765499597
PR Review Comment: https://git.openjdk.org/jdk/pull/21045#discussion_r1765502560


More information about the client-libs-dev mailing list