Author ezio.melotti
Recipients Oleg.Plakhotnyuk, ezio.melotti, r.david.murray, vstinner
Date 2011-12-07.16:56:00
SpamBayes Score 1.10984e-05
Marked as misclassified No
Message-id <1323276961.27.0.241175232406.issue13394@psf.upfronthosting.co.za>
In-reply-to
Content
I think the patch includes too many changes.  It would be better if you can split it in two separate patches: one with the increased coverage, the other with the bugs you found and their tests.
I can easily commit the first (assuming it doesn't break the test suite), but the second requires someone familiar with the aifc module for a review.
If you have found (and fixed) several unrelated bugs in aifc, it might be better to split the second patch further, and possibly open different issues, stating what the bugs are and how you fixed them.
History
Date User Action Args
2011-12-07 16:56:01ezio.melottisetrecipients: + ezio.melotti, vstinner, r.david.murray, Oleg.Plakhotnyuk
2011-12-07 16:56:01ezio.melottisetmessageid: <1323276961.27.0.241175232406.issue13394@psf.upfronthosting.co.za>
2011-12-07 16:56:00ezio.melottilinkissue13394 messages
2011-12-07 16:56:00ezio.melotticreate