On Mon, Apr 28, 2008 at 2:05 PM, <[EMAIL PROTECTED]> wrote:

> Fri Apr 25 16:01:53 EDT 2008  [EMAIL PROTECTED]
>  * fpstring.c: switch a memchr for memrchr
>  See <http://bugs.darcs.net/issue814>; memrchr speeds up is_funky quite a
> bit and thus helps whatsnew -s. It doesn't seem to break (any more) tests.


Now that we're looking at less of the data does memrchr really provide a
speed up?  I'm also concerned about what Don Stewart said about the
portability of memrchr.  I don't see a portability fix included in your
patch bundle.  So, my recommendation would be to wait on the memrchr
addition.

The rest looks good.

Jason
_______________________________________________
darcs-users mailing list
[email protected]
http://lists.osuosl.org/mailman/listinfo/darcs-users

Reply via email to