Hi Karen,
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.
You are welcome.
During jprt testing there was an issue with some debugging code I had in
there, this turned
out to be a problem on Windows, as I was not closing the
DirectoryStream. I have removed
that code and the updated webrev is here:
http://cr.openjdk.java.net/~ksrini/8022547/webrev.1/
to make it easier, the delta webrev showing only the changes since the
last is here:
http://cr.openjdk.java.net/~ksrini/8022547/webrev.1/webrev.delta/index.html
I presume the push will also delete the file from the current location :-)
Of course :-) , however since those are in langtools repository and
already in ignored state,
I will get that done separately.
Thanks
Kumar
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