RFR(S): JDK-8016644 - Improve UnsupportedClassVersionError message

[Christian Tornqvist]

Hi Chris,

 

This is one of those tests that doesn't need any of the extra options, we're
simply verifying the error message. There are several cases where we
shouldn't pass the flags and I think this might be one of them.

 

Thanks,

Christian

 

From: Christian Thalinger [mailto:christian.thalinger at oracle.com] 
Sent: Wednesday, February 5, 2014 11:03 PM
To: Christian Tornqvist
Cc: hotspot-runtime-dev at openjdk.java.net
Subject: Re: RFR(S): JDK-8016644 - Improve UnsupportedClassVersionError
message

 

Not so much about this particular change but the way testing is done.  I
don't really like how we are spawning other Java processes:

  41         ProcessBuilder pb =
ProcessTools.createJavaProcessBuilder("-cp", ".",  "ClassFile");

As far as I can see there is no code that passes on the flags which are
defined with either -vmoption or JAVA_VM_ARGS.  If I tell the testing
framework to use a particular flag then I expect all Java processes to use
them.

 

Can we please change that?

 

On Feb 5, 2014, at 7:19 PM, Christian Tornqvist
<christian.tornqvist at oracle.com <mailto:christian.tornqvist at oracle.com> >
wrote:





Hi everyone,

 

This change is about making the UnsupportedClassVersionError message a bit
more user friendly.

 

Webrev can be found at:

 <http://cr.openjdk.java.net/~ctornqvi/webrev/8016644/webrev.00/>
http://cr.openjdk.java.net/~ctornqvi/webrev/8016644/webrev.00/

 

Bug:

 <https://bugs.openjdk.java.net/browse/JDK-8016644>
https://bugs.openjdk.java.net/browse/JDK-8016644

 

Thanks,

Christian

 

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.openjdk.java.net/pipermail/hotspot-runtime-dev/attachments/20140206/e208fc60/attachment.html