comparison contrib/perf.py @ 40537: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

40536:1d3bed7d2923

        beginrev = startrev
        endrev = rllen
        dist = opts[b'dist']

        if reverse:
            beginrev, endrev = endrev, beginrev
            dist = -1 * dist

        for x in _xrange(beginrev, endrev, dist):
            # Old revisions don't support passing int.
            n = rl.node(x)

40537:cbd251d479bb

        beginrev = startrev
        endrev = rllen
        dist = opts[b'dist']

        if reverse:
            beginrev, endrev = endrev - 1, beginrev - 1
            dist = -1 * dist

        for x in _xrange(beginrev, endrev, dist):
            # Old revisions don't support passing int.
            n = rl.node(x)