Raymond Hettinger <raymond.hettin...@gmail.com> added the comment:

Nice work.  There are a few nits (dealloc and traverse need to address both 
python objects in the struct).  I will look at the patch in detail when I get a 
chance (in the next week or so).

It would be great if more tests were added (in general, C equivalents require 
more testing than their pure Python counterparts).

If you get a chance, it would be great if we could see some timings to 
demonstrate that we're getting a significant improvement (that being the 
primary motivation for the patch).

----------

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

Reply via email to