Patchwork [08,of,13] perf: run 'setup' function during stub run

login
register
mail settings
Submitter Boris Feld
Date Nov. 23, 2018, 2:09 p.m.
Message ID <1ab87863dc79a8e3139c.1542982144@localhost.localdomain>
Download mbox | patch
Permalink /patch/36737/
State Accepted
Headers show

Comments

Boris Feld - Nov. 23, 2018, 2:09 p.m.
# HG changeset patch
# User Boris Feld <boris.feld@octobus.net>
# Date 1542926924 -3600
#      Thu Nov 22 23:48:44 2018 +0100
# Node ID 1ab87863dc79a8e3139c723e83bef8c1ce107016
# Parent  e2599c3fba4223ced20a7a253e67631851cab700
# EXP-Topic perf-branchmap
# Available At https://bitbucket.org/octobus/mercurial-devel/
#              hg pull https://bitbucket.org/octobus/mercurial-devel/ -r 1ab87863dc79
perf: run 'setup' function during stub run

The benchmarked function might need the content of the setup to be run in order
to function properly.

Patch

diff --git a/contrib/perf.py b/contrib/perf.py
--- a/contrib/perf.py
+++ b/contrib/perf.py
@@ -276,6 +276,8 @@  def gettimer(ui, opts=None):
     return functools.partial(_timer, fm, displayall=displayall), fm
 
 def stub_timer(fm, func, setup=None, title=None):
+    if setup is not None:
+        setup()
     func()
 
 @contextlib.contextmanager