[PATCH 4 of 4 lazy-manifest] context: don't sort manifest entries

Gregory Szorc gregory.szorc at gmail.com
Fri Jan 9 03:11:27 CST 2015



> On Jan 8, 2015, at 12:35, Augie Fackler <raf at durin42.com> wrote:
> 
> # HG changeset patch
> # User Augie Fackler <augie at google.com>
> # Date 1416200425 18000
> #      Mon Nov 17 00:00:25 2014 -0500
> # Node ID 1bb05f6482bc53f6f1ca5851bd9238ef1570c156
> # Parent  07da98fd38a1269c899b3c81fbe23dbc807511d0
> context: don't sort manifest entries
> 
> The manifest iterator is now pre-sorted, so we can skip this check.
> 
> diff --git a/mercurial/context.py b/mercurial/context.py
> --- a/mercurial/context.py
> +++ b/mercurial/context.py
> @@ -101,8 +101,7 @@ class basectx(object):
>         return self.filectx(key)
> 
>     def __iter__(self):
> -        for f in sorted(self._manifest):
> -            yield f
> +        return (f for f in self._manifest)

return iter(self_manifest)


More information about the Mercurial-devel mailing list