This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author eli.bendersky
Recipients Arfrever, Robin.Schreiber, asvetlov, effbot, eli.bendersky, pitrou
Date 2013-01-13.22:52:35
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1358117556.65.0.0169954753285.issue15651@psf.upfronthosting.co.za>
In-reply-to
Content
I looked at the patch a bit more in depth and must admit that I'm reluctant to apply it. It's a very large patch with very little documentation about what steps are taken and why, and I just don't see the motivation. 

The way I see it, PEP 384 is great for compatibility of third-party extensions and embeddings of Python, but much less critical for a module that's always distributed as part of stdlib and thus is kept in exact sync with the ABI of the Python version it comes with. Correct me if I'm wrong.

That said, I won't object to some refactoring if it improves the code. But when such large changes are proposed, I really prefer to see small, incremental patches that replace just a part of the code. Such patches should come with an explanation of why the change is made (i.e. which part of PEP 384 does it adhere to).
History
Date User Action Args
2013-01-13 22:52:36eli.benderskysetrecipients: + eli.bendersky, effbot, pitrou, Arfrever, asvetlov, Robin.Schreiber
2013-01-13 22:52:36eli.benderskysetmessageid: <1358117556.65.0.0169954753285.issue15651@psf.upfronthosting.co.za>
2013-01-13 22:52:36eli.benderskylinkissue15651 messages
2013-01-13 22:52:35eli.benderskycreate