[PATCH] scmutil: speed up revrange

Bryan O'Sullivan bos at serpentine.com
Mon Apr 9 23:59:52 CDT 2012


On Mon, Apr 9, 2012 at 9:01 PM, Bryan O'Sullivan <bos at serpentine.com> wrote:

> +                if not seen and not l:
> +                    # by far the most common path
> +                    l = range(start, end + step, step)
> +                    continue
>

This should be updating seen, too. Strangely, no tests fail, nor can I
construct a command line that misbehaves without the added "seen = set(l)".

But in any case, I'll send an updated patch.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://selenic.com/pipermail/mercurial-devel/attachments/20120409/ac22dc37/attachment.html>


More information about the Mercurial-devel mailing list