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

login
register
mail settings
Submitter phabricator
Date Aug. 31, 2019, 10:59 p.m.
Message ID <347e7737757637e31670590a19028c08@localhost.localdomain>
Download mbox | patch
Permalink /patch/41451/
State Not Applicable
Headers show

Comments

phabricator - Aug. 31, 2019, 10:59 p.m.
Closed by commit rHG3f81d58aae25: statprof: clarify by naming tuple members while enumerate()&#039;ing (authored by martinvonz).
This revision was automatically updated to reflect the committed changes.

REPOSITORY
  rHG Mercurial

CHANGES SINCE LAST UPDATE
  https://phab.mercurial-scm.org/D6778?vs=16343&id=16348

CHANGES SINCE LAST ACTION
  https://phab.mercurial-scm.org/D6778/new/

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

AFFECTED FILES
  mercurial/statprof.py

CHANGE DETAILS




To: martinvonz, #hg-reviewers, pulkit
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)