[issue13394] Patch to increase aifc lib test coverage with couple of minor fixes

Ezio Melotti report at bugs.python.org
Wed Dec 7 17:56:01 CET 2011


Ezio Melotti <ezio.melotti at 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 <report at bugs.python.org>
<http://bugs.python.org/issue13394>
_______________________________________


More information about the Python-bugs-list mailing list