Write files directly from yaml.dump
authorJeremy Stanley <fungi@yuggoth.org>
Sat, 17 Jan 2015 01:16:13 +0000 (01:16 +0000)
committerJeremy Stanley <fungi@yuggoth.org>
Sat, 17 Jan 2015 01:16:13 +0000 (01:16 +0000)
Simplify data file writing by passing a file descriptor directly to
the yaml.dump method rather than unnecessarily passing a string back
out of it and into a separate writer.

lib/mudpy/data.py

index ca47f83..b3ca5a8 100644 (file)
@@ -4,7 +4,6 @@
 # to use, copy, modify, and distribute this software is granted under
 # terms provided in the LICENSE file distributed with this software.
 
-import codecs
 import os
 import re
 import stat
@@ -144,7 +143,7 @@ class DataFile:
                     os.rename(self.filename, self.filename + ".0")
 
             # our data file
-            file_descriptor = codecs.open(self.filename, "w", "utf-8")
+            file_descriptor = open(self.filename, "w")
 
             # if it's marked private, chmod it appropriately
             if self.filename in self.universe.private_files and oct(
@@ -152,9 +151,9 @@ class DataFile:
                ) != 0o0600:
                 os.chmod(self.filename, 0o0600)
 
-            # write, flush and close the file
-            file_descriptor.write(yaml.dump(self.data))
-            file_descriptor.flush()
+            # write and close the file
+            yaml.dump(self.data, allow_unicode=True, default_flow_style=False,
+                      stream=file_descriptor)
             file_descriptor.close()
 
             # unset the modified flag