Hi Greg

Thanks for the review.

Greg Brown wrote:
> Line 68: - Why not use an ArrayList for the variable item heights? Also, this 
> member should be renamed to "itemHeights", and its declaration should be 
> moved below editIndex (it is currently declared between variableItemHeight 
> and checkboxPadding, which implies that it is a style).
>
>   
Because I was using ArrayList, and you said to change it to an array in
the last review.

Applied the rest of the formatting, renaming and moving suggestions.


-- Noel.

Reply via email to