Re: Review request for 8001536 updated

2012-11-01 Thread Alan Bateman
On 31/10/2012 15:08, Lance Andersen - Oracle wrote: Here is revised webrev taking into account Remi's suggestions http://cr.openjdk.java.net/~lancea/8001536/webrev.01/ I skimmed over the updated webrev and the changes mostly look okay to me. One comment on the clone method is that "The inter

Re: Review request for 8001536 updated

2012-10-31 Thread Remi Forax
On 10/31/2012 04:08 PM, Lance Andersen - Oracle wrote: Here is revised webrev taking into account Remi's suggestions http://cr.openjdk.java.net/~lancea/8001536/webrev.01/ looks good :) Best, Lance RĂ©mi On Oct 30, 2012, at 2:05 PM, Remi Forax wrote: On 10/30/2012 05:25 PM, Lance Anders

Re: Review request for 8001536 updated

2012-10-31 Thread Lance Andersen - Oracle
Here is revised webrev taking into account Remi's suggestions http://cr.openjdk.java.net/~lancea/8001536/webrev.01/ Best, Lance On Oct 30, 2012, at 2:05 PM, Remi Forax wrote: > On 10/30/2012 05:25 PM, Lance Andersen - Oracle wrote: >> Hi, >> >> This is a request for review of >> http://cr.o