X-Git-Url: https://git.phdru.name/?a=blobdiff_plain;f=mimedecode.py;h=9db427893301d1455214134f1d6cbddde0d6738a;hb=2c4b69435e4b6ac5d3a5eb78f7ca86405132f7ca;hp=b6f139b6974ba2cb141d9c6999085cfdce22a02a;hpb=4467e66adb26be6409c68e7ff5408776b224170a;p=mimedecode.git

diff --git a/mimedecode.py b/mimedecode.py
index b6f139b..9db4278 100755
--- a/mimedecode.py
+++ b/mimedecode.py
@@ -6,11 +6,6 @@ from mimedecode_version import __version__, __author__, __copyright__, __license
 import sys, os
 import email
 
-try:
-    from cStringIO import StringIO
-except ImportError:
-    from StringIO import StringIO
-
 me = os.path.basename(sys.argv[0])
 
 
@@ -30,9 +25,6 @@ Usage: %s [-h|--help] [-V|--version] [-cCDP] [-H|--host=hostname] [-f charset] [
     sys.exit(code)
 
 
-def output(s):
-    gopts.outfile.write(s)
-
 def output_headers(msg):
     unix_from = msg.get_unixfrom()
     if unix_from:
@@ -188,12 +180,6 @@ def recode_charset(msg, s):
 def totext(msg, instring):
     "Convert instring content to text"
 
-    if msg.is_multipart(): # Recursively decode all parts of the multipart message
-        newfile = StringIO(msg.as_string())
-        newfile.seek(0)
-        decode_file(newfile)
-        return
-
     # Decode body and recode charset
     s = decode_body(msg, instring)
     if gopts.recode_charset:
@@ -245,11 +231,8 @@ def decode_part(msg):
     totext(msg, outstring)
 
 
-def decode_file(infile):
-    "Decode the entire message"
-
-    msg = email.message_from_file(infile)
-    boundary = msg.get_boundary()
+def decode_message(msg):
+    "Decode message"
 
     if msg.is_multipart():
         decode_headers(msg)
@@ -258,21 +241,28 @@ def decode_file(infile):
         if msg.preamble: # Preserve the first part, it is probably not a RFC822-message
             output(msg.preamble) # Usually it is just a few lines of text (MIME warning)
 
+        boundary = msg.get_boundary()
+
         for subpart in msg.get_payload():
-            output("\n--%s\n" % boundary)
-            decode_part(subpart)
+            if boundary:
+                output("\n--%s\n" % boundary)
 
-        output("\n--%s--\n" % boundary)
+            if subpart.is_multipart(): # Recursively decode all parts of the subpart
+                decode_message(subpart)
+            else:
+                decode_part(subpart)
+
+        if boundary:
+            output("\n--%s--\n" % boundary)
 
         if msg.epilogue:
             output(msg.epilogue)
 
-    else:
-        if msg.has_key("Content-Type"): # Simple one-part message - decode it
-            decode_part(msg)
+    elif msg.has_key("Content-Type"): # Simple one-part message - decode it
+        decode_part(msg)
 
-        else: # Not a message, just text - copy it literally
-            output(msg.as_string())
+    else: # Not a message, just text - copy it literally
+        output(msg.as_string())
 
 
 class GlobalOptions:
@@ -391,7 +381,10 @@ if __name__ == "__main__":
         gopts.host_name = socket.gethostname()
 
     gopts.outfile = outfile
-    decode_file(infile)
+    output = outfile.write
 
-    infile.close()
-    outfile.close()
+    try:
+        decode_message(email.message_from_file(infile))
+    finally:
+        infile.close()
+        outfile.close()