[Biopython-dev] [Bug 3016] Change WriterTests in test_PhyloXML.py to use StringIO or temp files

[bugzilla-daemon at portal.open-bio.org]

http://bugzilla.open-bio.org/show_bug.cgi?id=3016





------- Comment #2 from chapmanb at 50mail.com  2010-05-19 07:35 EST -------
Eric;
Just a quick tip on mkstemp. When you do:

DUMMY = tempfile.mkstemp()[1]

you leave an open handle as the first argument of this tuple. It won't cause
you any issues here, but is a problem if you have a long running server
process. You will leak open file handles and eventually get an error about too
many open files. See:

http://www.logilab.org/blogentry/17873
http://vocamus.net/dave/?p=997

No problems here, but rather a heads up on a tricky bit of python I've run into
too many times to count,
Brad


-- 
Configure bugmail: http://bugzilla.open-bio.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.