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

Coleen Phillimore coleen.phillimore at oracle.com
Mon May 20 11:31:51 PDT 2013


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

>
>> PS: Aleksey, I am stating process to make you Hotspot Committer (HSX
>> Author first) because you have already 9 changesets in our sources:
> Awesome, thanks.
>
> -Aleksey.



More information about the hotspot-dev mailing list