intermediary variable isn't needed

it's sufficiently clear this way, and uses less code.

diff --git a/edward-bot b/edward-bot
index 15d53280eec11ced2f3f38ed38abd0c925e45330..7e68f9a92c7d481ffc538fb9f30f0d5007330b9c 100755 (executable)
--- a/edward-bot
+++ b/edward-bot
@@ -35,12 +35,18 @@ import re
 import io
 import time
 
+
 def main ():
 
     handle_args()
 
-    txt = sys.stdin.read()
-    msg = email.parser.Parser().parsestr(txt)
+    email_text = sys.stdin.read()
+    print(decode_email(email_text))
+
+
+def decode_email (text):
+
+    msg = email.parser.Parser().parsestr(text)
 
     message  = ""
     message += "From: " + msg['From'] + "\n"
@@ -48,7 +54,7 @@ def main ():
 
     message += msg_walk(msg)
 
-    print(message)
+    return message
 
 
 def msg_walk (msg):
@@ -73,8 +79,7 @@ def msg_walk (msg):
 
 
         if (filename == "encrypted.asc") or (conttype == "pgp/mime"):
-            payload_dec = decrypt_payload(payload)
-            body += payload_dec
+            body += decrypt_payload(payload)
 
         elif conttype == "text/plain":
             body += payload + "\n"
@@ -84,6 +89,7 @@ def msg_walk (msg):
 
     return body
 
+
 def get_part_info (part):
 
     charset   = part.get_content_charset()