You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by gg...@apache.org on 2014/10/15 15:57:41 UTC
svn commit: r1632037 -
/commons/proper/email/trunk/src/main/java/org/apache/commons/mail/HtmlEmail.java
Author: ggregory
Date: Wed Oct 15 13:57:40 2014
New Revision: 1632037
URL: http://svn.apache.org/r1632037
Log:
Statement unnecessarily nested within else clause.
Modified:
commons/proper/email/trunk/src/main/java/org/apache/commons/mail/HtmlEmail.java
Modified: commons/proper/email/trunk/src/main/java/org/apache/commons/mail/HtmlEmail.java
URL: http://svn.apache.org/viewvc/commons/proper/email/trunk/src/main/java/org/apache/commons/mail/HtmlEmail.java?rev=1632037&r1=1632036&r2=1632037&view=diff
==============================================================================
--- commons/proper/email/trunk/src/main/java/org/apache/commons/mail/HtmlEmail.java (original)
+++ commons/proper/email/trunk/src/main/java/org/apache/commons/mail/HtmlEmail.java Wed Oct 15 13:57:40 2014
@@ -274,12 +274,9 @@ public class HtmlEmail extends MultiPart
{
return ii.getCid();
}
- else
- {
- throw new EmailException("embedded name '" + name
- + "' is already bound to URL " + urlDataSource.getURL()
- + "; existing names cannot be rebound");
- }
+ throw new EmailException("embedded name '" + name
+ + "' is already bound to URL " + urlDataSource.getURL()
+ + "; existing names cannot be rebound");
}
// verify that the URL is valid