On 6/8/18 12:27 PM, mandy chung wrote:

Webrev:
http://cr.openjdk.java.net/~bchristi/8204565/webrev/


test/jdk/java/lang/System/Versions.java
   it can also verify java.vm.specification.version.

The hotspot test looks to me that it should expect the test be run
with OpenJDK build and the vendor verification should consider that.
That's a separate issue unrelated to this change.  I suggest to
remove the comment at line 36.

Otherwise, looks good.

OK, thanks - done. (Also updated the @bug, copyright year, and removed unused 'VMversion'.

-Brent

Reply via email to