On Wed, Sep 8, 2010 at 6:12 PM, Ian Lynagh <igloo@earth.li> wrote:

copyI describes its last parameter as
First offset in source /not/ to copy (i.e. /not/ length)
but I think it's actually the first offset in /dest/ not to /write to/.

That's correct.

In Data.Text.append there's a (len1+len2) that could be just len.

Fixed :)

This program segfaults (on 64bit; try 2^28 if on 32bit):

Also fixed, and a regression test added.

Thanks!