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 ezio.melotti
Recipients belopolsky, benjamin.peterson, brandon-rhodes, daniel.urban, eric.araujo, ezio.melotti, ncoghlan, pitrou, sandro.tosi
Date 2011-08-04.22:59:08
SpamBayes Score 2.0384805e-07
Marked as misclassified No
Message-id <1312498749.43.0.693225777354.issue11572@psf.upfronthosting.co.za>
In-reply-to
Content
ISTM that the patch is trying to do too many things at once:
1) increase the test coverage, possibly fixing some bugs discovered while doing so;
2) refactor the tests to use the correct assert methods;
3) get rid of old code, and do some refactoring in copy.py;

I'm not sure any of the changes in copy.py is necessary to make the test suite pass, even after the additions you included in the patch (I haven't tested though).  If this is the case, the refactoring/cleanup of copy.py should IMHO be committed separately.
For the tests it's probably fine to commit both the additions and the refactoring together (i.e., it's not worth wasting time splitting the patch).
History
Date User Action Args
2011-08-04 22:59:09ezio.melottisetrecipients: + ezio.melotti, ncoghlan, belopolsky, pitrou, benjamin.peterson, eric.araujo, daniel.urban, sandro.tosi, brandon-rhodes
2011-08-04 22:59:09ezio.melottisetmessageid: <1312498749.43.0.693225777354.issue11572@psf.upfronthosting.co.za>
2011-08-04 22:59:08ezio.melottilinkissue11572 messages
2011-08-04 22:59:08ezio.melotticreate