RFR (S) CR 8014878: Clean up class field layout code

Aleksey Shipilev aleksey.shipilev at oracle.com
Mon May 20 12:46:30 PDT 2013


On 05/20/2013 10:31 PM, Coleen Phillimore wrote:
> 
> On 05/20/2013 01:33 PM, Aleksey Shipilev wrote:
>> On 05/20/2013 08:29 PM, Vladimir Kozlov wrote:
>>> Looks good.
>> Thanks for the review!
>>
>>> Could you fix indentions in lines 3176 (move 1 space left), 3572 (move 1
>>> space right)?
>> Done:
>>   http://cr.openjdk.java.net/~shade/8014878/webrev.02/
>>
>> We still need a second reviewer, right?
>> Coleen, I bet you are anxious to see that method being refactored.
> 
> This looks good and helpful.

Thanks for the review!

The changeset is here:
  http://cr.openjdk.java.net/~shade/8014878/8014878.01.changeset

Please sponsor :)

-Aleksey.



More information about the hotspot-dev mailing list