mercurial/scmutil.py
changeset 25385 a26a55406c0a
parent 25384 99d3ca7d67e4
child 25386 a5a95642144b
--- a/mercurial/scmutil.py	Sun May 24 17:59:55 2015 +0900
+++ b/mercurial/scmutil.py	Sun May 24 18:11:33 2015 +0900
@@ -709,7 +709,7 @@
             return defval
         return repo[val].rev()
 
-    l = revset.baseset([])
+    subsets = []
 
     revsetaliases = [alias for (alias, _) in
                      repo.ui.configitems("revsetalias")]
@@ -725,7 +725,7 @@
                 raise error.RepoLookupError
 
             if isinstance(spec, int):
-                l = l + revset.baseset([spec])
+                subsets.append(revset.baseset([spec]))
                 continue
 
             if _revrangesep in spec:
@@ -738,27 +738,21 @@
                 if end == nullrev and start < 0:
                     start = nullrev
                 rangeiter = repo.changelog.revs(start, end)
-                if not l:
-                    # by far the most common case: revs = ["-1:0"]
-                    l = revset.baseset(rangeiter)
-                    continue
-                l = l + revset.baseset(rangeiter)
+                l = revset.baseset(rangeiter)
+                subsets.append(l)
                 continue
             elif spec and spec in repo: # single unquoted rev
                 rev = revfix(repo, spec, None)
-                l = l + revset.baseset([rev])
+                subsets.append(revset.baseset([rev]))
                 continue
         except error.RepoLookupError:
             pass
 
         # fall through to new-style queries if old-style fails
         m = revset.match(repo.ui, spec, repo)
-        if l:
-            l = l + m(repo)
-        else:
-            l = m(repo)
+        subsets.append(m(repo))
 
-    return l
+    return revset._combinesets(subsets)
 
 def expandpats(pats):
     '''Expand bare globs when running on windows.