Tim Peters <t...@python.org> added the comment:

Yes, it's better to have tp_clear than not for a variety of reasons (including 
setting examples of best practice).

Best I can tell, the patch for BPO-33418 was reverted _only_ to worm around the 
crash in _this_ report.  That's no longer needed.  Or, if it is, we've missed 
something fundamental in the fix for this bug.  In which case, reverting the 
other patch is the only clear way to find that out quickly.  I favor getting it 
right.

----------

_______________________________________
Python tracker <rep...@bugs.python.org>
<https://bugs.python.org/issue38006>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to