Request for review: 7064927: retransformClasses() does not pass in LocalVariableTable of a method

[Paul Hohensee]

Looks good.

Thanks,

Paul

On 12/21/11 3:35 PM, Thomas Wuerthinger wrote:
> Thanks for the review. I've corrected the comments and created a new 
> webrev at: http://cr.openjdk.java.net/~thomaswue/7064927/webrev.00/
>
> - thomas
>
>
> On 20.12.2011 23:09, Paul Hohensee wrote:
>> Looks good, except that the LocalVariableTable attribute comments in
>> write_code_attribute() and write_local_variable_table_attribute() don't
>> match.  The comment in the latter says "LineNumberTable" instead
>> of "LocalVariableTable" and the descriptor is for the line number table
>> rather than the local variable table.  The comment in the former is 
>> missing
>> "local_variable_table[local_variable_table_length];"
>>
>> Paul
>>
>> On 12/20/11 4:26 PM, Thomas Wuerthinger wrote:
>>> Hi!
>>>
>>> I'd like to resend my request for review and kindly ask for a review 
>>> for my patch that fixes bug number 7064927.
>>> The webrev is available at: 
>>> http://cr.openjdk.java.net/~coleenp/7064927/
>>>
>>> Thanks, thomas
>>>
>>>
>>> On 14.11.2011 12:18, Thomas Wuerthinger wrote:
>>>> Hi!
>>>>
>>>> An open webrev of the patch that fixes bug 7064927 is available at: 
>>>> http://cr.openjdk.java.net/~coleenp/7064927/
>>>> The bug is described at: 
>>>> http://bugs.sun.com/view_bug.do?bug_id=7064927
>>>>
>>>> Thanks, thomas
>>>
>