[PATCH 2 of 8 filtering] clfilter: remove usage of `range` and `xrange` in `scmutil.revrange`

Pierre-Yves David pierre-yves.david at ens-lyon.org
Tue Nov 20 18:11:34 CST 2012


# HG changeset patch
# User Pierre-Yves David <pierre-yves.david at ens-lyon.org>
# Date 1353434746 -3600
# Node ID 80f2f2c90a15af59fa9175b6520d7659acc7bb15
# Parent  deb7a59af85a0827bc5ea7501bb1ea58a1752e69
clfilter: remove usage of `range` and `xrange` in `scmutil.revrange`

For changelog level filtering to take effect it need to be used for any
iteration.

This changeset removes usage of `range` and `xrange` that survived the first
pass.

diff --git a/mercurial/scmutil.py b/mercurial/scmutil.py
--- a/mercurial/scmutil.py
+++ b/mercurial/scmutil.py
@@ -632,17 +632,17 @@ def revrange(repo, revs):
 
             if _revrangesep in spec:
                 start, end = spec.split(_revrangesep, 1)
                 start = revfix(repo, start, 0)
                 end = revfix(repo, end, len(repo) - 1)
-                step = start > end and -1 or 1
+                rangeiter = repo.changelog.revs(start, end)
                 if not seen and not l:
                     # by far the most common case: revs = ["-1:0"]
-                    l = range(start, end + step, step)
+                    l = list(rangeiter)
                     # defer syncing seen until next iteration
                     continue
-                newrevs = set(xrange(start, end + step, step))
+                newrevs = set(rangeiter)
                 if seen:
                     newrevs.difference_update(seen)
                     seen.update(newrevs)
                 else:
                     seen = newrevs


More information about the Mercurial-devel mailing list