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 r.david.murray
Recipients Aman.Shah, Arfrever, alexandre.vassalotti, pitrou, r.david.murray, serhiy.storchaka
Date 2013-02-28.22:11:41
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1362089501.54.0.271954437121.issue17299@psf.upfronthosting.co.za>
In-reply-to
Content
Aman: another nit: PEP8 calls for no unneeded parentheses around logical expressions, so things like:

   if(self.f):

should be written:

   if self.f:

in order to adhere to our coding style.

Also, it's not obvious to me that there is any reason to rename the base classes (PicklerTests->AbstractIOPicklerTests, and same for Persistent test).

Finally, rereading Serhiy's note, Python3 only has "general IO streams", both non-IO files and cStringIO.StringIO are gone.  So this really is a 2.7-only issue, and only worth doing because 2.7 is a long term maintenance release.
History
Date User Action Args
2013-02-28 22:11:41r.david.murraysetrecipients: + r.david.murray, pitrou, alexandre.vassalotti, Arfrever, serhiy.storchaka, Aman.Shah
2013-02-28 22:11:41r.david.murraysetmessageid: <1362089501.54.0.271954437121.issue17299@psf.upfronthosting.co.za>
2013-02-28 22:11:41r.david.murraylinkissue17299 messages
2013-02-28 22:11:41r.david.murraycreate