RFR: 8022547: [verifier] move DefaultMethodRegressionTests.java to jdk

Karen Kinnear karen.kinnear at oracle.com
Fri Aug 9 20:06:28 UTC 2013


Kumar,

Looks good. And thank you - we definitely needed the URLClassLoader so that we
actually test the verifier, unless we had a force -Xverify:all flag. Thank you for
fixing the major version from 0x33 ->0x34.

I presume the push will also delete the file from the current location :-)

thanks,
Karen

On Aug 7, 2013, at 8:25 PM, Kumar Srinivasan wrote:

> Hello,
> 
> Testing this functionality in langtools does not seem to be the appropriate location,
> and the teams have decided to move it to jdk/test/vm/verifier, which seems to be the
> logical place.
> 
> I have modified the test to remove testng dependencies, also added a variant which
> involves loading classes via an URLClassLoader which exposed a test bug wrt. the
> class.version.major not being correct, therefore it is useful to have this variant anyway.
> 
> The webrev:
> http://cr.openjdk.java.net/~ksrini/8022547/webrev.0/
> 
> The bug:
> http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=8022547
> 
> Thanks
> Kumar
> 
> 
> 
> 




More information about the core-libs-dev mailing list