[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Chicken-hackers] [PATCH] Avoid using / in xsubstring and string-xco

From: Mario Domenech Goulart
Subject: Re: [Chicken-hackers] [PATCH] Avoid using / in xsubstring and string-xcopy!, since srfi-13 is compiled with (declare (fixnum))
Date: Fri, 29 Jun 2012 14:56:07 -0400
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/23.1.91 (gnu/linux)

On Fri, 29 Jun 2012 20:49:49 +0200 Peter Bex <address@hidden> wrote:

> On Thu, Jun 28, 2012 at 10:25:11AM -0400, Mario Domenech Goulart wrote:
>> The attached patch contains a fix to #869
>> (
>> srfi-13.scm is compiled with (declare (fixnum)), but `xsubstring' and
>> `string-xcopy!' use / in a cond clause as a shorter path to avoid
>> calling `%multispan-repcopy!', which is more "expensive".
>> The patch just removes that shorter path, so the fixnum declaration can
>> be kept.
> Excellent, I've pushed this,

Thank you.

> but I still think we should consider pulling in the tests.  Now we
> just have this fix without a regression test, it might creep back in
> if we ever decide to rewrite parts of SRFI-13 (as its performance
> really is suboptimal in general we probably want to consider doing
> this some day...)

You are right.  I'm gonna port the BSD-licensed part of
(from Gauche) and provide a patch.

Best wishes.

reply via email to

[Prev in Thread] Current Thread [Next in Thread]