Patchwork [03,of,13] perf: use an explicit function in perfbranchmapload

login
register
mail settings
Submitter Boris Feld
Date Nov. 23, 2018, 2:08 p.m.
Message ID <56efcdd74aee7b45d2b8.1542982139@localhost.localdomain>
Download mbox | patch
Permalink /patch/36731/
State Accepted
Headers show

Comments

Boris Feld - Nov. 23, 2018, 2:08 p.m.
# HG changeset patch
# User Boris Feld <boris.feld@octobus.net>
# Date 1542933177 -3600
#      Fri Nov 23 01:32:57 2018 +0100
# Node ID 56efcdd74aee7b45d2b85a7d414033cae6b465c7
# Parent  9f29b499e0adb22a3fd23f8e88ea281d1073aa3f
# EXP-Topic perf-branchmap
# Available At https://bitbucket.org/octobus/mercurial-devel/
#              hg pull https://bitbucket.org/octobus/mercurial-devel/ -r 56efcdd74aee
perf: use an explicit function in perfbranchmapload

This make things clearer.

Patch

diff --git a/contrib/perf.py b/contrib/perf.py
--- a/contrib/perf.py
+++ b/contrib/perf.py
@@ -2199,7 +2199,9 @@  def perfbranchmapload(ui, repo, filter=b
         raise error.Abort(b'No branchmap cached for %s repo'
                           % (filter or b'unfiltered'))
     timer, fm = gettimer(ui, opts)
-    timer(lambda: branchmap.read(repo) and None)
+    def bench():
+        branchmap.read(repo)
+    timer(bench)
     fm.end()
 
 @command(b'perfloadmarkers')