Close files earlier

diff --git a/babel/localedata.py b/babel/localedata.py
index 9c6f9f2b955a0efd7767e3c985a23265aed2e941..87981fe2a574e2c52b7ad687be4aebc657f76588 100644 (file)
--- a/babel/localedata.py
+++ b/babel/localedata.py
@@ -112,9 +112,9 @@ def load(name, merge_inherited=True):
                     merge(data, pickle.load(fileobj))
                 else:
                     data = pickle.load(fileobj)
-                _cache[name] = data
             finally:
                 fileobj.close()
+            _cache[name] = data
         return data
     finally:
         _cache_lock.release()

diff --git a/babel/messages/frontend.py b/babel/messages/frontend.py
index 89a7b54ab913b7d4fd616e64de67fb938d8afe1c..a4e9c2e577105dd6af7d8e7d999e0482ac068f68 100644 (file)
--- a/babel/messages/frontend.py
+++ b/babel/messages/frontend.py
@@ -474,10 +474,10 @@ class extract_messages(Command):
             fileobj = open(self.mapping_file, 'U')
             try:
                 method_map, options_map = parse_mapping(fileobj)
-                for path in self.input_paths:
-                    mappings.append((path, method_map, options_map))
             finally:
                 fileobj.close()
+            for path in self.input_paths:
+                mappings.append((path, method_map, options_map))
 
         elif getattr(self.distribution, 'message_extractors', None):
             message_extractors = self.distribution.message_extractors