This is an archive of the discontinued Mercurial Phabricator instance.

Differential D6443

py3: use range() instead of xrange()
ClosedPublic

Authored by pulkit on May 24 2019, 8:41 AM.

Details

Reviewers
indygreg
Group Reviewers
hg-reviewers
Commits
rHGc2d10506725c: py3: use range() instead of xrange()
Summary

The latter does not exist on Python 3. This makes test-contrib-perf.t pass on
Python 3 again.

Diff Detail

Repository
rHG Mercurial
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

pulkit created this revision.May 24 2019, 8:41 AM
indygreg accepted this revision.May 24 2019, 11:19 PM
This revision is now accepted and ready to land.May 24 2019, 11:19 PM
Closed by commit rHGc2d10506725c: py3: use range() instead of xrange() (authored by pulkit). · Explain WhyMay 24 2019, 11:21 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Mcontrib/perf.py (2 lines)

Diff 15257

contrib/perf.py

def _timer(fm, func, setup=None, title=None, displayall=False, def _timer(fm, func, setup=None, title=None, displayall=False,
limits=DEFAULTLIMITS, prerun=0, profiler=None): limits=DEFAULTLIMITS, prerun=0, profiler=None):
gc.collect() gc.collect()
results = [] results = []
begin = util.timer() begin = util.timer()
count = 0 count = 0
if profiler is None: if profiler is None:
profiler = NOOPCTX profiler = NOOPCTX
for i in xrange(prerun): for i in range(prerun):
if setup is not None: if setup is not None:
setup() setup()
func() func()
keepgoing = True keepgoing = True
while keepgoing: while keepgoing:
if setup is not None: if setup is not None:
setup() setup()
with profiler: with profiler: