Review request for JDK-8019632: Method parameters are not copied in clone_with_new_data

Coleen Phillmore coleen.phillimore at oracle.com
Thu Jul 18 08:22:10 PDT 2013


Eric,
This code change looks good.   For the related bug that didn't copy 
method annotations, I added test: 
jdk/test/java/lang/instrument/RedefineMethodWithAnnotations.sh. Can you 
use this test to write a new one to the JDK that demonstrates this bug?  
We've been adding these tests to the JDK because it has the support 
there and all the other tests are there.

Thanks,
Coleen

On 7/18/2013 9:38 AM, Eric McCorkle wrote:
> Realized I'd duplicated a comment.  It's been corrected.
>
> On 07/17/13 23:06, Eric McCorkle wrote:
>> Hello,
>>
>> Please review this patch, which updates clone_with_new_data to copy
>> method parameter data.  This was missed in the initial implementation.
>>
>> I've gotten a JPRT run that failed only due to running out of memory on
>> some tests, and I'll be doing my usual complete testlist run soon.
>>
>> The webrev is here:
>> http://cr.openjdk.java.net/~emc/8019632/
>>
>> The bug report is here:
>> http://bugs.sun.com/view_bug.do?bug_id=8019632
>>
>> Thanks,
>> Eric
>>



More information about the hotspot-dev mailing list