On Feb 19, 2012, at 7:11 PM, Rafael Ávila de Espíndola 
<[email protected]> wrote:

> Nico noticed that the patch included an infinite loop from when I
> renamed merge2 to mergeWithMin.
> 
> An updated patch is attached.

I think this is ok. Some more comments on the visibility that should be 
happening in the various testcases and where you use mergeWithMin would be 
helpful next time I need to review something through there.

-eric
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to