mirror of https://github.com/python/cpython
Merged revisions 83531 via svnmerge from
svn+ssh://pythondev@svn.python.org/python/branches/py3k ........ r83531 | georg.brandl | 2010-08-02 19:24:49 +0200 (Mo, 02 Aug 2010) | 1 line #7372: fix regression in pstats: a previous fix to handle cProfile data in add_callers broke handling of profile data. ........
This commit is contained in:
parent
58f55d5d42
commit
a2a92cf2c0
|
@ -513,8 +513,13 @@ def add_callers(target, source):
|
|||
new_callers[func] = caller
|
||||
for func, caller in source.iteritems():
|
||||
if func in new_callers:
|
||||
if isinstance(caller, tuple):
|
||||
# format used by cProfile
|
||||
new_callers[func] = tuple([i[0] + i[1] for i in
|
||||
zip(caller, new_callers[func])])
|
||||
else:
|
||||
# format used by profile
|
||||
new_callers[func] += caller
|
||||
else:
|
||||
new_callers[func] = caller
|
||||
return new_callers
|
||||
|
|
|
@ -10,10 +10,16 @@ class AddCallersTestCase(unittest.TestCase):
|
|||
def test_combine_results(self):
|
||||
"""pstats.add_callers should combine the call results of both target
|
||||
and source by adding the call time. See issue1269."""
|
||||
# new format: used by the cProfile module
|
||||
target = {"a": (1, 2, 3, 4)}
|
||||
source = {"a": (1, 2, 3, 4), "b": (5, 6, 7, 8)}
|
||||
new_callers = pstats.add_callers(target, source)
|
||||
self.assertEqual(new_callers, {'a': (2, 4, 6, 8), 'b': (5, 6, 7, 8)})
|
||||
# old format: used by the profile module
|
||||
target = {"a": 1}
|
||||
source = {"a": 1, "b": 5}
|
||||
new_callers = pstats.add_callers(target, source)
|
||||
self.assertEqual(new_callers, {'a': 2, 'b': 5})
|
||||
|
||||
|
||||
def test_main():
|
||||
|
|
|
@ -26,6 +26,9 @@ Library
|
|||
|
||||
- Issue #9354: Provide getsockopt() in asyncore's file_wrapper.
|
||||
|
||||
- Issue #7372: Fix pstats regression when stripping paths from profile
|
||||
data generated with the profile module.
|
||||
|
||||
- Issue #4108: In urllib.robotparser, if there are multiple 'User-agent: *'
|
||||
entries, consider the first one.
|
||||
|
||||
|
|
Loading…
Reference in New Issue