The fix looks good to me.
Thanks,
Alexandr.
On 14/04/16 11:41, Yuri Nesterenko wrote:
Ok, +1, simplest working device is better, definitely in the tests.
-yan
On 04/14/2016 10:05 AM, Semyon Sadetsky wrote:
On 4/13/2016 9:57 PM, Philip Race wrote:
Perhaps better but given the minimal use here can't we just
replace it with
System.getProperty("os.name").toLowerCase().contains("windows") ?
Then you neither need jtreg to run it, nor internal API.
I agree. See the updated webrev:
http://cr.openjdk.java.net/~ssadetsky/8153276/webrev.01/
--Semyon
-phil.
On 4/13/16, 11:44 AM, Sergey Bylokhov wrote:
+1
On 13.04.16 17:17, Yuri Nesterenko wrote:
Looks good to me.
-yan
On 04/13/2016 05:07 PM, Semyon Sadetsky wrote:
Hello,
Please review fix for JDK9:
bug: https://bugs.openjdk.java.net/browse/JDK-8153276
webrev: http://cr.openjdk.java.net/~ssadetsky/8153276/webrev.00/
Internal API usage sun.awt.OSInfo was replaced by
jdk.testlibrary.OSInfo.
--Semyon
