Message 124444 - Python tracker

➜

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	pitrou
Recipients	Chris.Gerhard, anacrolix, christian.heimes, daniel.urban, loewis, pitrou, rosslagerwall
Date	2010-12-21.18:28:42
SpamBayes Score	4.9872664e-08
Marked as misclassified	No
Message-id	<[email protected]>
In-reply-to

Content
Thanks for the patch. A couple of comments: - the C code is misindented in some places (using 8 spaces rather than 4) - you should use support.unlink consistently in the tests (rather than sometimes os.unlink or posix.unlink) - when cleaning up in tests (through unlink() or rmdir()), it's better to use finally clauses so that cleaning up gets done even on error; or, alternatively, to use self.addCleanup() (see http://docs.python.org/dev/library/unittest.html#unittest.TestCase.addCleanup) (I haven't looked at the C code in detail since you say it's mostly copy/paste from existing code)

Thanks for the patch. A couple of comments:
- the C code is misindented in some places (using 8 spaces rather than 4)
- you should use support.unlink consistently in the tests (rather than sometimes os.unlink or posix.unlink)
- when cleaning up in tests (through unlink() or rmdir()), it's better to use finally clauses so that cleaning up gets done even on error; or, alternatively, to use self.addCleanup() (see http://docs.python.org/dev/library/unittest.html#unittest.TestCase.addCleanup)

(I haven't looked at the C code in detail since you say it's mostly copy/paste from existing code)

History
Date	User	Action	Args
2010-12-21 18:28:46	pitrou	set	recipients: + pitrou, loewis, christian.heimes, daniel.urban, anacrolix, Chris.Gerhard, rosslagerwall
2010-12-21 18:28:46	pitrou	set	messageid: <[email protected]>
2010-12-21 18:28:42	pitrou	link	issue4761 messages
2010-12-21 18:28:42	pitrou	create