Brian Thorne <hardb...@gmail.com> added the comment:

I've updated the patch to address the comments here and in the code review.

I added more cross testing of the pure Python implementation of partial - as 
you pointed out inheritance wasn't supported so I changed from the simple 
closure to a class implementation.

Instead of skipping repr tests for the pure Python implementation could we not 
just implement it? I did skip the pickle test for the Python implementation 
though.

Nick, I wasn't sure how to decorate the partial object as a staticmethod so I 
don't think this patch addresses issue 11704.

Also I didn't understand why Lock was being imported from _thread instead of 
thread. Since coverage went to 100% and the tests continue to all pass when I 
changed this.

----------
Added file: http://bugs.python.org/file25185/12428.patch

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

Reply via email to