➜

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 Aman.Shah
Recipients Aman.Shah, Arfrever, alexandre.vassalotti, pitrou, r.david.murray, serhiy.storchaka
Date 2013-03-01.07:27:09
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1362122829.58.0.478016783742.issue17299@psf.upfronthosting.co.za>
In-reply-to
Content 
I had put in the renaming because "ioclass" is not defined in the class itself, only in the subclasses. So, instantiating it and using dumps/loads would be meaningless and would raise an error. Which is similar to the behavior of an abstract class.
Also, the "if(self.f):" part can be improved as you said and I looked at the patch for other similar mistakes. But, it's an isolated mistake (at lines 33 and 74) and so maybe it can be fixed during the merge itself?
History
Date User Action Args
2013-03-01 07:27:09Aman.Shahsetrecipients: + Aman.Shah, pitrou, alexandre.vassalotti, Arfrever, r.david.murray, serhiy.storchaka
2013-03-01 07:27:09Aman.Shahsetmessageid: <1362122829.58.0.478016783742.issue17299@psf.upfronthosting.co.za>
2013-03-01 07:27:09Aman.Shahlinkissue17299 messages
2013-03-01 07:27:09Aman.Shahcreate