RFR 8010293 <was> Re: Potential issue with CHM.toArray

Chris Hegarty chris.hegarty at oracle.com
Fri Sep 6 14:43:23 UTC 2013


On 06/09/2013 12:08, Paul Sandoz wrote:
>
> ....
>>   http://hg.openjdk.java.net/lambda/lambda/jdk/rev/b73937e96ae0
>>   http://hg.openjdk.java.net/lambda/lambda/jdk/raw-rev/b73937e96ae0
>>
>
> Anyone up for reviewing this?

I skimmed the patch and nothing jumps out at me.

There does appear to be some code duplication for:
   void pushState(Node<K,V>[] t, int i, int n) {
   void recoverState(int n) {

   , but it may not be easily refactored.

I'm fine to be listed as reviewer, if you need one.

-Chris.

>
> Thanks,
> Paul.



More information about the core-libs-dev mailing list