Author: ggregory
Date: Wed Oct 15 13:57:55 2014
New Revision: 1632038

URL: http://svn.apache.org/r1632038
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=1632038&r1=1632037&r2=1632038&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:55 2014
@@ -390,12 +390,9 @@ public class HtmlEmail extends MultiPart
             {
                 return ii.getCid();
             }
-            else
-            {
-                throw new EmailException("embedded name '" + file.getName()
-                    + "' is already bound to file " + existingFilePath
-                    + "; existing names cannot be rebound");
-            }
+            throw new EmailException("embedded name '" + file.getName()
+                + "' is already bound to file " + existingFilePath
+                + "; existing names cannot be rebound");
         }
 
         // verify that the file is valid


Reply via email to