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.00The test needs to check for the supported platform first and exit gracefully if it is run on an unsupported one.Thanks, -JB-