Mercurial > public > mercurial-scm > hg-stable
comparison mercurial/logcmdutil.py @ 42335:1ce46f0ee218
log: remove an unnecessary "and opts.get('rev')" condition
As Yuya pointed out, the condition is unnecessary since
revs.isdescending() would be true if --follow without --rev.
Differential Revision: https://phab.mercurial-scm.org/D6372
| author | Martin von Zweigbergk <martinvonz@google.com> |
|---|---|
| date | Tue, 14 May 2019 09:13:39 -0700 |
| parents | db26dbbee74d |
| children | 604c086ddde6 |
comparison
equal
deleted
inserted
replaced
| 42334:264a2cbb25d0 | 42335:1ce46f0ee218 |
|---|---|
| 741 if filematcher is None: | 741 if filematcher is None: |
| 742 def filematcher(ctx): | 742 def filematcher(ctx): |
| 743 return match | 743 return match |
| 744 | 744 |
| 745 expr = _makerevset(repo, match, pats, slowpath, opts) | 745 expr = _makerevset(repo, match, pats, slowpath, opts) |
| 746 if opts.get('graph') and opts.get('rev'): | 746 if opts.get('graph'): |
| 747 # User-specified revs might be unsorted, but don't sort before | 747 # User-specified revs might be unsorted, but don't sort before |
| 748 # _makerevset because it might depend on the order of revs | 748 # _makerevset because it might depend on the order of revs |
| 749 if not (revs.isdescending() or revs.istopo()): | 749 if not (revs.isdescending() or revs.istopo()): |
| 750 revs.sort(reverse=True) | 750 revs.sort(reverse=True) |
| 751 if expr: | 751 if expr: |