[PATCH] multipart mime output

"P. A. Bagyenda" <[email protected]>
Newsgroups gmane.comp.mobile.kannel.devel
Message-ID <[email protected]>
Hi,
    Is there any good reason why we add an extra CRLF to the end of   
a message part? The patch attached proposes to reverse this, as it  
seems to expand the message size each time we go from mime -> octstr - 
 > mime. Any objections?

P.
mime.diff (application/octet-stream, 588 B)
Index: gwlib/mime.c
===================================================================
RCS file: /home/cvs/gateway/gwlib/mime.c,v
retrieving revision 1.20
diff -u -r1.20 mime.c
--- gwlib/mime.c	18 Apr 2007 08:11:29 -0000	1.20
+++ gwlib/mime.c	17 Jul 2007 12:42:27 -0000
@@ -271,7 +271,7 @@
 
     /* add the last boundary statement, but hive an EOL 
      * if we are on the top of the recursion stack. */
-    /* if (level > 0) */
+    if (level > 0) 
         octstr_append(mime, octstr_imm("\r\n"));
     octstr_append(mime, octstr_imm("\r\n--"));
     octstr_append(mime, boundary);
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.