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 pitrou
Recipients pitrou, sbt, tim.peters, vstinner
Date 2013-10-08.17:32:55
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1381253573.2582.2.camel@fsol>
In-reply-to <1381251493.4.0.801188289609.issue19158@psf.upfronthosting.co.za>
Content
Le mardi 08 octobre 2013 à 16:58 +0000, Tim Peters a écrit :
> Tim Peters added the comment:
> 
> This is the "right" way to do it:  the subclass wants to extend the
> behavior of the base class .release(), not to replace it.  Calling the
> base class .release() is the natural and obvious way to do that.

Well... IMOHO it would be "natural and obvious" if those were in
different modules written by different developers.
Here I think that "practicality beats purity", though.
(although I'm not sure the person who originally wrote that sentence can
really be trusted ;-))

> About acquire(), this is a bugfix - I'm not looking here to change
> code that isn't broken ;-)

Agreed. Just wanted to mention it before I'd totally forget about it.
History
Date User Action Args
2013-10-08 17:32:55pitrousetrecipients: + pitrou, tim.peters, vstinner, sbt
2013-10-08 17:32:55pitroulinkissue19158 messages
2013-10-08 17:32:55pitroucreate