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

Aleksey Shipilev aleksey.shipilev at oracle.com
Mon May 20 10:33:41 PDT 2013


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.

> 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