[PATCH 1 of 3] scmutil: changed revrange to return lazysets for new style revsets

Lucas Moscovicz lmoscovicz at fb.com
Sat Feb 22 13:19:32 CST 2014


# HG changeset patch
# User Lucas Moscovicz <lmoscovicz at fb.com>
# Date 1392247827 28800
#      Wed Feb 12 15:30:27 2014 -0800
# Node ID 857bff65281118fee59b3d636835409925cabafc
# Parent  be5b87453bdb33ea6d11d50b832d909224b746a3
scmutil: changed revrange to return lazysets for new style revsets

When there is an old style revset or both it will still return a baseset. This
may be changed in later patches.

diff --git a/mercurial/scmutil.py b/mercurial/scmutil.py
--- a/mercurial/scmutil.py
+++ b/mercurial/scmutil.py
@@ -489,7 +489,7 @@
         try:
             if isinstance(spec, int):
                 seen.add(spec)
-                l.append(spec)
+                l = l + [spec]
                 continue
 
             if _revrangesep in spec:
@@ -510,7 +510,7 @@
                     seen.update(newrevs)
                 else:
                     seen = newrevs
-                l.extend(sorted(newrevs, reverse=start > end))
+                l = l + sorted(newrevs, reverse=start > end)
                 continue
             elif spec and spec in repo: # single unquoted rev
                 rev = revfix(repo, spec, None)
@@ -524,11 +524,14 @@
 
         # fall through to new-style queries if old-style fails
         m = revset.match(repo.ui, spec)
-        dl = [r for r in m(repo, revset.spanset(repo)) if r not in seen]
-        l.extend(dl)
-        seen.update(dl)
+        if seen or l:
+            dl = [r for r in m(repo, revset.spanset(repo)) if r not in seen]
+            l = l + dl
+            seen.update(dl)
+        else:
+            l = m(repo, revset.spanset(repo))
 
-    return revset.baseset(l)
+    return l
 
 def expandpats(pats):
     if not util.expandglobs:


More information about the Mercurial-devel mailing list