Review request: JDK-8062556: Add jdk tests for JDK-8058322 and JDK-8058313

Eric McCorkle eric.mccorkle at oracle.com
Fri Oct 31 14:01:09 UTC 2014


I went through and added comments in the binary data indicating where
the MethodParameters attributes are, and a breakdown of their contents.
 I went ahead and did this for all the bad class files, not just the new
ones.

There is a larger picture here: there's an outstanding task I filed
around the time these tests were written to find a better way for
langtools to run jtreg tests that involve bad class files.
Unfortunately, doing that is rather difficult, as you can see.  The only
real way to do it is to generate a class file, convert it to signed
bytes (you can't even use hex; you get an unsigned/signed byte
conversion problem), then modify the data by hand.  The intent is to
replace this with a better method at some point.

On 10/30/14 21:59, David Holmes wrote:
> Hi Erik,
> 
> On 31/10/2014 9:41 AM, Eric McCorkle wrote:
>> Hello,
>>
>> Please review this patch which adds tests to the JDK test suite for two
>> reflection bugs that require hotspot changes (JDK-8058322 and
>> JDK-8058313)
>>
>> The webrev is here:
>> http://cr.openjdk.java.net/~emc/8062556/
> 
> I second Brian's comment re the source of the bad classes.
> 
> Your webrev is broken btw - no top-level html files.
> 
> The new test needs a copyright year of 2014 not 2013.
> 
> Thanks,
> David
> 



More information about the core-libs-dev mailing list