systematic use of context managers for dealing with files instead of open()/close...

[nepi.git] / src / nepi / util / serializer.py

diff --git a/src/nepi/util/serializer.py b/src/nepi/util/serializer.py
index ad7a4c3..0e36a92 100644 (file)
--- a/src/nepi/util/serializer.py
+++ b/src/nepi/util/serializer.py
@@ -28,9 +28,8 @@ class ECSerializer(object):
             from nepi.util.parsers.xml_parser import ECXMLParser
             
             parser = ECXMLParser()
-            f = open(filepath, "r")
-            xml = f.read()
-            f.close()
+            with open(filepath, "r") as f:
+                xml = f.read()
 
             ec = parser.from_xml(xml)
 
@@ -52,9 +51,8 @@ class ECSerializer(object):
         if format == SFormats.XML:
             filepath = os.path.join(dirpath, "%s.xml" % filename)
             sec = self.serialize(ec, format = format)
-            f = open(filepath, "w")
-            f.write(sec)
-            f.close()
+            with open(filepath, "w") as f:
+                f.write(sec)
 
         return filepath