summary refs log tree commit diff
diff options
context:
space:
mode:
authorSergey M․ <dstftw@gmail.com>2014-08-21 22:03:00 +0700
committerSergey M․ <dstftw@gmail.com>2014-08-21 22:03:00 +0700
commit73159f99cc3be44b207154ecd56d16750cade77a (patch)
tree957a9262c50d625d7a65c9b2fcf07e6b5f2a3c6c
parent55db73efdf1860bbd24b8556fbefd2ba6263a2cf (diff)
downloadyoutube-dl-73159f99cc3be44b207154ecd56d16750cade77a.tar.gz
youtube-dl-73159f99cc3be44b207154ecd56d16750cade77a.tar.xz
youtube-dl-73159f99cc3be44b207154ecd56d16750cade77a.zip
[utils] Add missing mode and encoding arguments
-rw-r--r--youtube_dl/utils.py22
1 files changed, 14 insertions, 8 deletions
diff --git a/youtube_dl/utils.py b/youtube_dl/utils.py
index d11e46c80..f8ec5389f 100644
--- a/youtube_dl/utils.py
+++ b/youtube_dl/utils.py
@@ -233,18 +233,24 @@ else:
 def write_json_file(obj, fn):
     """ Encode obj as JSON and write it to fn, atomically """
 
+    args = {
+        'suffix': '.tmp',
+        'prefix': os.path.basename(fn) + '.',
+        'dir': os.path.dirname(fn),
+        'delete': False,
+    }
+
     # In Python 2.x, json.dump expects a bytestream.
     # In Python 3.x, it writes to a character stream
     if sys.version_info < (3, 0):
-        mode = 'wb'
-        encoding = None
+        args['mode'] = 'wb'
     else:
-        mode = 'w'
-        encoding = 'utf-8'
-    tf = tempfile.NamedTemporaryFile(
-        suffix='.tmp', prefix=os.path.basename(fn) + '.',
-        dir=os.path.dirname(fn),
-        delete=False)
+        args.update({
+            'mode': 'w',
+            'encoding': 'utf-8',
+        })
+
+    tf = tempfile.NamedTemporaryFile(**args)
 
     try:
         with tf: