Skip to content
Snippets Groups Projects
Commit 4c6b2076 authored by Gregory Szorc's avatar Gregory Szorc
Browse files

perf: move revlog construction and length calculation out of benchmark

We don't need to measure the time it takes to open the revlog or
calculate its length.

This is more consistent with what other perf* functions do.

While I was here, I also renamed the revlog variable from "r" to
"rl" - again in the name of consistency.
parent 954f6339
No related branches found
No related tags found
No related merge requests found
......@@ -850,6 +850,7 @@
The start revision can be defined via ``-s/--startrev``.
"""
_len = getlen(ui)
rl = cmdutil.openrevlog(repo, 'perfrevlog', file_, opts)
rllen = getlen(ui)(rl)
def d():
......@@ -854,6 +855,5 @@
def d():
r = cmdutil.openrevlog(repo, 'perfrevlog', file_, opts)
r.clearcaches()
rl.clearcaches()
beginrev = startrev
......@@ -858,6 +858,6 @@
beginrev = startrev
endrev = _len(r)
endrev = rllen
dist = opts['dist']
if reverse:
......@@ -865,7 +865,7 @@
dist = -1 * dist
for x in xrange(beginrev, endrev, dist):
r.revision(x)
rl.revision(x)
timer, fm = gettimer(ui, opts)
timer(d)
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment