Repository navigation
Broken compatibility in FrameSummary equality #69298
Description
| BPO | 25111 |
|---|---|
| Nosy | @vstinner, @rbtcollins, @berkerpeksag, @serhiy-storchaka |
| Files |
Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.
Show more details
GitHub fields:
assignee = 'https://github.com/serhiy-storchaka'
closed_at = <Date 2015-09-29.19:37:09.222>
created_at = <Date 2015-09-14.17:48:43.752>
labels = ['type-bug', 'library']
title = 'Broken compatibility in FrameSummary equality'
updated_at = <Date 2015-09-29.19:37:09.221>
user = 'https://github.com/serhiy-storchaka'bugs.python.org fields:
activity = <Date 2015-09-29.19:37:09.221>
actor = 'serhiy.storchaka'
assignee = 'serhiy.storchaka'
closed = True
closed_date = <Date 2015-09-29.19:37:09.222>
closer = 'serhiy.storchaka'
components = ['Library (Lib)']
creation = <Date 2015-09-14.17:48:43.752>
creator = 'serhiy.storchaka'
dependencies = []
files = ['40463', '40500']
hgrepos = []
issue_num = 25111
keywords = ['patch']
message_count = 7.0
messages = ['250693', '250953', '251722', '251760', '251834', '251835', '251880']
nosy_count = 5.0
nosy_names = ['vstinner', 'rbcollins', 'python-dev', 'berker.peksag', 'serhiy.storchaka']
pr_nums = []
priority = 'normal'
resolution = 'fixed'
stage = 'resolved'
status = 'closed'
superseder = None
type = 'behavior'
url = 'https://bugs.python.org/issue25111'
versions = ['Python 3.5', 'Python 3.6']
Activity
serhiy-storchaka commentedon Sep 14, 2015
Since 3.5 traceback.extract_tb() and traceback.extract_stack() return a list (actually list's subclass StackSummary) of FrameSummary objects instead of a list of tuples. FrameSummary is not fully compatible with tuple. One important incompatibility is in the comparing.
In general __eq__ shouldn't raise an exception. bpo-25108 is needed to be fixed first for tests.
Here is a patch that restores compatibility.
An alternative solution would be to make FrameSummary a subclass of tuple (namedtuple).
serhiy-storchaka commentedon Sep 18, 2015
Fixed patch and added tests.
serhiy-storchaka commentedon Sep 27, 2015
Ping.
berkerpeksag commentedon Sep 28, 2015
LGTM
+ self.assertEqual(f, tuple(f))
+ self.assertEqual(tuple(f), f)
It would be good to add a comment to explain why we test both variants, but it's not really important.
vstinner commentedon Sep 29, 2015
traceback_FrameSummary_equality_2.patch looks good to me, but I have the same remark than Berker (see the review).
rbtcollins commentedon Sep 29, 2015
LGTM too.
python-dev commentedon Sep 29, 2015
New changeset 2ecb7d4d9e0b by Serhiy Storchaka in branch '3.5':
Issue bpo-25111: Fixed comparison of traceback.FrameSummary.
https://hg.python.org/cpython/rev/2ecb7d4d9e0b
New changeset f043182a81d7 by Serhiy Storchaka in branch 'default':
Issue bpo-25111: Fixed comparison of traceback.FrameSummary.
https://hg.python.org/cpython/rev/f043182a81d7