<Swing Dev> JDK9 Review Request for 8138881: Bug in OSInfo.java

Sergey Bylokhov Sergey.Bylokhov at oracle.com
Wed Oct 14 15:13:52 UTC 2015


Hi, Rajeev.
The fix looks fine, a few comments about the test:
  - Please use some useful name instead of bugNumber.
  - @test @bug should be in the different rows.
  - This test uses classes from the "sun.awt." package, which means that 
soon by default it will not be available to the test. You need to add:
    * @modules java.desktop/sun.awt
       You can check it on the latest jigsaw builds:
       https://jdk9.java.net/jigsaw/

On 14.10.15 15:51, Rajeev Chamyal wrote:
> Hello,
>
> Please review the following fix for Jdk9:
>
> WebRev : http://cr.openjdk.java.net/~rchamyal/8138881/webrev.00/
>
> Bug Id: https://bugs.openjdk.java.net/browse/JDK-8138881
>
> Issue: There was a typo in OSInfo.java.
>
> Regards,
>
> Rajeev Chamyal
>


-- 
Best regards, Sergey.



More information about the swing-dev mailing list