Re: [matplotlib-devel] Filehandle leaks in font_manager.py (was: PDF backend problem)

2007-03-13 Thread Edin Salkovic
I would just suggest using open() instead of file(), but this is a general Python tip :) Everything else seems OK. Best, Edin On 3/13/07, Jouni K. Seppänen <[EMAIL PROTECTED]> wrote: > [Moving to the devel list.] > > Here's a patch for font_manager.py to close opened files. Could > somebody revi

[matplotlib-devel] Filehandle leaks in font_manager.py (was: PDF backend problem)

2007-03-13 Thread Jouni K . Seppänen
[Moving to the devel list.] Here's a patch for font_manager.py to close opened files. Could somebody review the patch? As I mentioned earlier, I don't understand why cPickle/pickle is not imported at the top level, so I'm hesitant to check this in without review. Index: lib/matplotlib/font_manage