helper: get rid of decode_string()

It just adds unnecessary type confusion.

In most places where it is used, the types are always str, so it does
nothing. In the few others, the encoding/decoding is better handled
explicitly.

1 files changed, 2 insertions, 3 deletions

diff --git a/alot/commands/envelope.py b/alot/commands/envelope.py
index 914829eb..1f806581 100644
--- a/alot/commands/envelope.py
+++ b/alot/commands/envelope.py
@@ -23,7 +23,6 @@ from .. import crypto
 from ..account import SendingMailFailed, StoreMailError
 from ..db.errors import DatabaseError
 from ..errors import GPGProblem
-from ..helper import string_decode
 from ..settings.const import settings
 from ..settings.errors import NoMatchingAccount
 from ..utils import argparse as cargparse
@@ -357,8 +356,8 @@ class EditCommand(Command):
             # get input
             # tempfile will be removed on buffer cleanup
             enc = settings.get('editor_writes_encoding')
-            with open(self.envelope.tmpfile.name) as f:
-                template = string_decode(f.read(), enc)
+            with open(self.envelope.tmpfile.name, 'rb') as f:
+                template = f.read().decode(enc)
 
             # call post-edit translate hook
             translate = settings.get_hook('post_edit_translate')