# HG changeset patch
# User Martin von Zweigbergk <martinvonz@google.com>
# Date 1632845323 25200
# Node ID b067d22dc6adc1c3fc7f0751d3f7d15873c08903
# Parent  73bcfde21fc25752d36d46834675fdc884b3da13
phase: avoid a no-op resolution of revset from revnums

I was surprised that `scmutil.revrange()` supports integers in the
list of revsets. I think it's clearer to not pass a list that's known
to contain only integers into the function.

Differential Revision: https://phab.mercurial-scm.org/D11559

diff -r 73bcfde21fc2 -r b067d22dc6ad mercurial/commands.py
--- a/mercurial/commands.py	Fri Oct 01 15:19:37 2021 +0200
+++ b/mercurial/commands.py	Tue Sep 28 09:08:43 2021 -0700
@@ -5219,13 +5219,13 @@
     # look for specified revision
     revs = list(revs)
     revs.extend(opts[b'rev'])
-    if not revs:
+    if revs:
+        revs = scmutil.revrange(repo, revs)
+    else:
         # display both parents as the second parent phase can influence
         # the phase of a merge commit
         revs = [c.rev() for c in repo[None].parents()]
 
-    revs = scmutil.revrange(repo, revs)
-
     ret = 0
     if targetphase is None:
         # display