it's possible to hit the finally with no f

This commit is contained in:
Chris Jones 2009-09-02 16:22:26 +01:00
parent 1d332c915e
commit b79c62ef49
1 changed files with 3 additions and 1 deletions

View File

@ -79,6 +79,7 @@ class Uninstall(Command):
return 'uninstall'
def run(self):
f = None
self.ensure_filename('manifest')
try:
try:
@ -87,7 +88,8 @@ class Uninstall(Command):
except IOError, e:
raise DistutilsFileError("unable to open install manifest: %s", str(e))
finally:
f.close()
if f:
f.close()
for file in files:
if os.path.isfile(file) or os.path.islink(file):