Review Request: 7140852: Add test for 7022100

[Stefan Karlsson]

On 2013-02-01 09:50, serguei.spitsyn at oracle.com wrote:
> Ship it.
Thanks!
StefanK

>
> Thanks,
> Serguei
>
>
> On 1/31/13 11:50 PM, Stefan Karlsson wrote:
>> On 2013-02-01 00:15, Coleen Phillimore wrote:
>>>
>>> Stefan,
>>>
>>> I just read through this test and it looks like a good test to me 
>>> (but I'm not an expert and it took a while to figure out how it 
>>> worked).   I had two questions.  Why does the same definition for 
>>> @interface ParameterAnnotation {} appear in both 
>>> RedefineMethodWithAnnotationTarget*.java files?   Can't it be in 
>>> it's own file and just once?  Or is it different (didn't see any 
>>> differences).
>> I've moved it to its own file now and added the needed extra 
>> infrastructure to get test to work with that.
>>
>>>
>>> Also is do_redefine supposed to be doRedefine as per Java coding 
>>> convention or is that a known variation?
>>
>> Fixed. It was the name used in the test that I copied the code from.
>>
>> http://cr.openjdk.java.net/~stefank/7140852/webrev.01/
>>
>> thanks,
>> StefanK
>>>
>>> Thanks,
>>> Coleen
>>>
>>> On 01/22/2013 09:39 AM, Stefan Karlsson wrote:
>>>> http://cr.openjdk.java.net/~stefank/7140852/webrev.00/
>>>>
>>>> This test provoked the bug in:
>>>> 7022100: Method annotations are incorrectly set when redefining 
>>>> classes
>>>>
>>>> thanks,
>>>> StefanK
>>>
>>
>