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 > > > >