finally in font_manager

The finally clauses in font_manager.py are python2.5 only and should
be removed. Michael, I suspect you added these when cleaning up the
open file handle bugs, but we'll have to find another solution.

Shouldn't these be picked up by the garbage collector?

JDH

John Hunter wrote:

The finally clauses in font_manager.py are python2.5 only and should
be removed. Michael, I suspect you added these when cleaning up the
open file handle bugs, but we'll have to find another solution.

I added this cleaning up a fairly narrow Vista-only bug at the suggestion of Richard Bolt, and without being on Vista myself :wink: (r4653).

http://sourceforge.net/tracker/index.php?func=detail&aid=1841933&group_id=80706&atid=560720

This change was made post-0.91.1, so I don't believe it made it into a release. I've changed it to nested "try"s as suggested in the Python reference docs.

Shouldn't these be picked up by the garbage collector?

Possibly, but the "finally" was there before I got there -- I assumed there was a need for it, and I didn't have Windows handy to test with.

Cheers,
Mike

ยทยทยท

--
Michael Droettboom
Science Software Branch
Operations and Engineering Division
Space Telescope Science Institute
Operated by AURA for NASA