Serhiy Storchaka added the comment:

I have added a lot of comments in Rietveld. In general the patch looks good, 
but I have some style nitpicks and some more strong comments. Matt, please 
update the patch.

Tests should test both Python and C implementations.

I doubt if get rid of locking is right. Hash calculation and comparison of 
arguments can call Python code and this code can recursive use the wrapped 
function.

Some benchmarks will be interesting.

I think this acceleration must necessarily be in 3.4.

----------
components: +Extension Modules -Interpreter Core
stage: patch review -> needs patch

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

Reply via email to