[Python-Dev] Py2.5 issue: decimal context manager misimplemented, misdesigned, and misdocumented

[Nick Coghlan]

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 at gmail.com   |   Brisbane, Australia
---------------------------------------------------------------
             http://www.boredomandlaziness.org