[darcs-devel] darcs patch: FastPackedString.hs: rw linesPS using By... (and 5 more)

[Gwern Branwen]

On Fri, Apr 11, 2008 at 10:34 AM, David Roundy <droundy at darcs.net> wrote:
> On Fri, Apr 11, 2008 at 12:06:43AM -0400, Gwern Branwen wrote:
>  >  Thu Apr 10 23:45:45 EDT 2008
>  >   * rm OldFastPackedString.hs
>
>  I'm rather confused.  Do you have some reason for eliminating the existing
>  non-buggy FastPackedString? Besides the fact that I don't have bytestring
>  installed, it seems like now is when it would be especially valuable to
>  you, since you're engaged in rewriting the current buggy bytestring code,
>  and it would be useful to have a correct reference to which you could
>  compare.

It was useful for making sure my binary was using FastPackedString.hs;
I kept a second darcs.net repo without my patches to test against
(since builds take so long, it was much more convenient to just go
'make install').

>  Could you resend without removing the existing implementation?
>  --
>  David Roundy

Certainly; it just got sent.

Anyway, what do you think of reverting your patch and making bytestring the
default? As Don says, this is a change that could use plenty of testing, and the
best people to do it are the ones working out of a darcs Darcs, I'd think.

--
gwern