diff contrib/perf.py @ 40587:cbd251d479bb

perf: fix perfrevlogrevisions --reverse Currently, 'endrev' equals `len(revlog)`, a revision that does not exist. When asking for the reverse order, the arguments passed to xrange are `xrange(len(revlog), startrev)` which then crash. We need to offset 'endrev' by one so we don't crash anymore. Also, we offset 'startrev' to ensure we get the same number of revisions with and without the `--reverse` option. Differential Revision: https://phab.mercurial-scm.org/D5228
author Boris Feld <boris.feld@octobus.net>
date Mon, 05 Nov 2018 17:24:39 +0100
parents e6c8a0fd3db4
children 164b2e77f9a5
line wrap: on
line diff
--- a/contrib/perf.py	Tue Nov 06 11:54:15 2018 +0100
+++ b/contrib/perf.py	Mon Nov 05 17:24:39 2018 +0100
@@ -1553,7 +1553,7 @@
         dist = opts[b'dist']
 
         if reverse:
-            beginrev, endrev = endrev, beginrev
+            beginrev, endrev = endrev - 1, beginrev - 1
             dist = -1 * dist
 
         for x in _xrange(beginrev, endrev, dist):