![](https://secure.gravatar.com/avatar/f3ba3ecffd20251d73749afbfa636786.jpg?s=120&d=mm&r=g)
Nick Coghlan wrote:
Raymond Hettinger wrote:
The right way to do it was presented in PEP343. The implementation was correct and the API was simple.
Raymond's persuaded me that he's right on the API part at the very least. The current API was a mechanical replacement of the initial __context__ based API with a normal method, whereas I should have reverted back to the module-level localcontext() function from PEP343 and thrown the method on Context objects away entirely.
I can fix it on the trunk (and add those missing tests!), but I'll need Anthony and/or Neal's permission to backport it and remove the get_manager() method from Python 2.5 before we get stuck with it forever.
I committed this fix as 51664 on the trunk (although the docstrings are still example free because doctest doesn't understand __future__ statements). Anthony, Neal: I'd like to backport this change to the 2.5 maintenance branch. I realise it is an API change between the release candidate and the actual release, but this really is a small tweak to something nobody is actually using yet. If that's not acceptable, I'd like to go with Raymond's original option: rip it out entirely for 2.5 so we don't get stuck maintaining it for the rest of the 2.x series. Cheers, Nick. -- Nick Coghlan | ncoghlan@gmail.com | Brisbane, Australia --------------------------------------------------------------- http://www.boredomandlaziness.org