Monotone-Parent: 7e8a884a4d8254949154e9c2650acb95f5c3565a

Monotone-Revision: 607aa602c2fc40fbaa85fb65c2f5b39d5ec5c3f6

Monotone-Author: wsourdeau@inverse.ca
Monotone-Date: 2009-06-04T18:56:14
Monotone-Branch: ca.inverse.sogo

SoObjects/Mailer/SOGoMailObject.m

@@ -567,6 +567,17 @@ static BOOL debugSoParts       = NO;
   body = [info objectForKey: @"body"];
   if (body)
     {
+      /* FIXME: this seems to generate bad mime part keys, which triggers a
+         exceptions such as this:
+
+         ERROR(-[NGImap4Client _processCommandParserException:]): catched
+         IMAP4 parser exception NGImap4ParserException: unsupported fetch key:
+         nil)
+
+         Do we really need to assign p to sp in a multipart body part? Or do
+         we need to do this only when the part in question is the first one in
+         the message? */
+
       sp = [[body valueForKey: @"type"] lowercaseString];
       if ([sp isEqualToString: @"multipart"])
 	sp = p;