Tim Peters added the comment:

All the timeout args are great!  I wish Python had always had them :-)

Back to the pain at hand, it's not the number of lines of code that rubs me the 
wrong way, but the sheer obscurity of it all.  This handful of lines is - of 
necessity - sprayed across two C code files, a C header file, and a Python 
module.  That makes it very hard (for anyone but you - LOL) to reconstruct the 
_intent_ of it all.

I'm adding another patch, which is your threadstate_join_2.patch but with a new 
comment block (in pystate.h) spelling out the rationale behind it all.  I can 
live with that ;-)

----------
Added file: http://bugs.python.org/file31635/threadstate_join_3.patch

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

Reply via email to