Patchwork D6778: statprof: clarify by naming tuple members while enumerate()'ing

login
register
mail settings
Submitter phabricator
Date Aug. 31, 2019, 5:35 p.m.
Message ID <differential-rev-PHID-DREV-jp4d22y2tf6bg7s5rijs-req@mercurial-scm.org>
Download mbox | patch
Permalink /patch/41446/
State Superseded
Headers show

Comments

phabricator - Aug. 31, 2019, 5:35 p.m.
martinvonz created this revision.
Herald added a subscriber: mercurial-devel.
Herald added a reviewer: hg-reviewers.

REPOSITORY
  rHG Mercurial

REVISION DETAIL
  https://phab.mercurial-scm.org/D6778

AFFECTED FILES
  mercurial/statprof.py

CHANGE DETAILS




To: martinvonz, #hg-reviewers
Cc: mercurial-devel

Patch

diff --git a/mercurial/statprof.py b/mercurial/statprof.py
--- a/mercurial/statprof.py
+++ b/mercurial/statprof.py
@@ -878,7 +878,8 @@ 
         laststack = collections.deque(stack)
     while laststack:
         poplast()
-    events = [s[1] for s in enumerate(samples) if s[0] not in blacklist]
+    events = [sample for idx, sample in enumerate(samples)
+              if idx not in blacklist]
     frames = collections.OrderedDict((str(k), v)
                                      for (k,v) in enumerate(id2stack))
     json.dump(dict(traceEvents=events, stackFrames=frames), fp, indent=1)