You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@commons.apache.org by di...@apache.org on 2007/02/26 01:42:36 UTC

svn commit: r511674 - /jakarta/commons/proper/email/trunk/src/java/org/apache/commons/mail/MultiPartEmail.java

Author: dion
Date: Sun Feb 25 16:42:35 2007
New Revision: 511674

URL: http://svn.apache.org/viewvc?view=rev&rev=511674
Log:
add todo for rethrow of exception

Modified:
    jakarta/commons/proper/email/trunk/src/java/org/apache/commons/mail/MultiPartEmail.java

Modified: jakarta/commons/proper/email/trunk/src/java/org/apache/commons/mail/MultiPartEmail.java
URL: http://svn.apache.org/viewvc/jakarta/commons/proper/email/trunk/src/java/org/apache/commons/mail/MultiPartEmail.java?view=diff&rev=511674&r1=511673&r2=511674
==============================================================================
--- jakarta/commons/proper/email/trunk/src/java/org/apache/commons/mail/MultiPartEmail.java (original)
+++ jakarta/commons/proper/email/trunk/src/java/org/apache/commons/mail/MultiPartEmail.java Sun Feb 25 16:42:35 2007
@@ -232,7 +232,7 @@
                 // the content for the main body part was actually set.  If not,
                 // an IOException will be thrown during super.send().
 
-                   BodyPart body = this.getPrimaryBodyPart();
+                BodyPart body = this.getPrimaryBodyPart();
                 try
                 {
                     body.getContent();
@@ -241,6 +241,8 @@
                 {
                     // do nothing here.  content will be set to an empty string
                     // as a result.
+                    // TODO: Should this reallyt be rethrown as an email exception:
+                    // throw new EmailException(e);
                 }
             }
 



---------------------------------------------------------------------
To unsubscribe, e-mail: commons-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: commons-dev-help@jakarta.apache.org