The fix looks OK.

Line 232 is not necessary.

The simplest fix might be only to add the following lines to 94
if (getPlatform() == null) {
   return;
}

Shanliang

Jaroslav Bachorik wrote:
Please, review this small test change.

Issue : https://bugs.openjdk.java.net/browse/JDK-8031420
Webrev: http://cr.openjdk.java.net/~jbachorik/8031420/webrev.00

The test needs to check for the supported platform first and exit gracefully if it is run on an unsupported one.

Thanks,

-JB-

Reply via email to