Mercurial > public > mercurial-scm > hg-stable
comparison mercurial/revset.py @ 37910:37e7ae332e90
revset: make id() an empty set for ambiguous nodeid (BC)
As Yuya pointed out in the review of D3311, id() (and rev()) does not
raise an error when the input is an unknown identifier, so it doesn't
make sense for it to do that when the input is ambiguous with a
filtered node. However, it turned out that it already does raise an
error when the input is ambiguous among the visible nodes. So let's
start by fixing that.
Differential Revision: https://phab.mercurial-scm.org/D3462
| author | Martin von Zweigbergk <martinvonz@google.com> |
|---|---|
| date | Mon, 07 May 2018 14:32:43 -0700 |
| parents | f83cb91b052e |
| children | 0a79fb64118e |
comparison
equal
deleted
inserted
replaced
| 37909:da083d9fafab | 37910:37e7ae332e90 |
|---|---|
| 1334 rn = None | 1334 rn = None |
| 1335 try: | 1335 try: |
| 1336 pm = repo.changelog._partialmatch(n) | 1336 pm = repo.changelog._partialmatch(n) |
| 1337 if pm is not None: | 1337 if pm is not None: |
| 1338 rn = repo.changelog.rev(pm) | 1338 rn = repo.changelog.rev(pm) |
| 1339 except LookupError: | |
| 1340 pass | |
| 1339 except error.WdirUnsupported: | 1341 except error.WdirUnsupported: |
| 1340 rn = node.wdirrev | 1342 rn = node.wdirrev |
| 1341 | 1343 |
| 1342 if rn is None: | 1344 if rn is None: |
| 1343 return baseset() | 1345 return baseset() |