Re: RFR: 8022547: [verifier] move DefaultMethodRegressionTests.java to jdk
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
Re: RFR: 8022547: [verifier] move DefaultMethodRegressionTests.java to jdk
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
RFR: 8022547: [verifier] move DefaultMethodRegressionTests.java to jdk
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