comparison mercurial/scmutil.py @ 41370:a728ef2f9b15

revpair: clarify check for empty revset Differential Revision: https://phab.mercurial-scm.org/D5688
author Martin von Zweigbergk <martinvonz@google.com>
date Thu, 24 Jan 2019 12:47:54 -0800
parents 5079242abef9
children b5169e79c31c
comparison
equal deleted inserted replaced
41369:5079242abef9 41370:a728ef2f9b15
670 if not revs: 670 if not revs:
671 return repo['.'], repo[None] 671 return repo['.'], repo[None]
672 672
673 l = revrange(repo, revs) 673 l = revrange(repo, revs)
674 674
675 if not l:
676 raise error.Abort(_('empty revision range'))
677
675 first = l.first() 678 first = l.first()
676 second = l.last() 679 second = l.last()
677 680
678 if first is None:
679 raise error.Abort(_('empty revision range'))
680 if (first == second and len(revs) >= 2 681 if (first == second and len(revs) >= 2
681 and not all(revrange(repo, [r]) for r in revs)): 682 and not all(revrange(repo, [r]) for r in revs)):
682 raise error.Abort(_('empty revision on one side of range')) 683 raise error.Abort(_('empty revision on one side of range'))
683 684
684 # if top-level is range expression, the result must always be a pair 685 # if top-level is range expression, the result must always be a pair