(resend) Re: Request for review (S): 7130974 G1: Remove G1ParCopyHelper

John Cuthbertson john.cuthbertson at oracle.com
Fri Mar 30 16:09:17 UTC 2012


Hi Bengt,

I'll take a look in a short while. Apologies - I had forgotten about this.

JohnC


On 03/30/12 06:22, Bengt Rutisson wrote:
>
> Hi all,
>
> Tony has reviewed this, but I still need one more a review. Any 
> takers? It's a pretty small change...
>
> Bengt
>
> On 2012-03-13 12:49, Bengt Rutisson wrote:
>>
>> Hi all,
>>
>> Could I have a couple of reviews of this small code clean up?
>>
>> http://cr.openjdk.java.net/~brutisso/7130974/webrev.00/
>>
>> Using templates it is possible to get rid of the extra inheritance 
>> level introduced by G1ParCopyHelper. This also removes a dependency 
>> between G1ParCopyHelper and G1ParCopyClosure regarding how the 
>> _scanner field is initialized.
>>
>> Bug report:
>> http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7130974
>>
>> Thanks,
>> Bengt
>




More information about the hotspot-gc-dev mailing list