On Mon, Feb 26, 2018 at 3:06 PM, mandy chung <mandy.ch...@oracle.com> wrote:
> > > On 2/21/18 12:30 PM, Martin Buchholz wrote: > line 63-64: ident further to the right as line 62 is the condition. > > Whitespace rejiggered. > Looks okay. I also think no need to have a separate copyToArrayDeque > method and just inline in the constructor. > It's used twice. Also, it's likely to be replaced someday when we decide what to do with lambdas, so good to keep as a separate method.