Ezio Melotti <ezio.melo...@gmail.com> added the comment: 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.
---------- nosy: +ezio.melotti _______________________________________ Python tracker <rep...@bugs.python.org> <http://bugs.python.org/issue13394> _______________________________________ _______________________________________________ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
