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

[Thomas Wuerthinger]

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
>>