RFR: 8022547: [verifier] move DefaultMethodRegressionTests.java to jdk
kumar.x.srinivasan at oracle.com
Sun Aug 11 00:25:06 UTC 2013
> 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:
to make it easier, the delta webrev showing only the changes since the
last is here:
> 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.
> On Aug 7, 2013, at 8:25 PM, Kumar Srinivasan wrote:
>> 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:
>> The bug:
More information about the core-libs-dev