revrange: attempt to parse old-style queries as a first pass

mercurial/cmdutil.py

@@ -149,14 +149,10 b' def revrange(repo, revs):'
 
     seen, l = set(), []
     for spec in revs:
-        if spec and not (
-            spec.startswith(revrangesep) or spec.endswith(revrangesep)):
-            m = revset.match(spec)
-            for r in m(repo, range(len(repo))):
-                if r not in seen:
-                    l.append(r)
-            seen.update(l)
-        elif revrangesep in spec:
+        # attempt to parse old-style ranges first to deal with
+        # things like old-tag which contain query metacharacters
+        try:
+            if revrangesep in spec:
             start, end = spec.split(revrangesep, 1)
             start = revfix(repo, start, 0)
             end = revfix(repo, end, len(repo) - 1)
@@ -166,12 +162,22 b' def revrange(repo, revs):'
                     continue
                 seen.add(rev)
                 l.append(rev)
-        else:
+                continue
+            elif spec in repo: # single unquoted rev
             rev = revfix(repo, spec, None)
             if rev in seen:
                 continue
             seen.add(rev)
             l.append(rev)
+        except error.RepoLookupError:
+            pass
+
+        # fall through to new-style queries if old-style fails
+        m = revset.match(spec)
+        for r in m(repo, range(len(repo))):
+            if r not in seen:
+                l.append(r)
+        seen.update(l)
 
     return l
 

tests/test-log.out

@@ -269,7 +269,7 b' diff --git a/b1 b/b1'
 +postm
 
 % log -r ""
-abort: 00changelog.i@: ambiguous identifier!
+hg: parse error: empty query
 % log -r <some unknown node id>
 abort: unknown revision '1000000000000000000000000000000000000000'!
 % log -k r1