> On 2 Dec 2015, at 17:16, Peter Levart <peter.lev...@gmail.com> wrote:
> 
> Hi Paul,
> 
> Just a nit more:
> 
> 120         int valuesPerWidth = LOG2_ARRAY_LONG_INDEX_SCALE - 
> log2ArrayIndexScale;
> 
> Would it be more correct to call that variable log2ValuesPerWidth?
> 

Yes, good point. Updated.  I also used the constant LOG2_ARRAY_INT_INDEX_SCALE 
instead of 2, here:

 138         if (log2ArrayIndexScale < LOG2_ARRAY_INT_INDEX_SCALE) {


Thanks,
Paul.

> Regards, Peter
> 

Reply via email to