You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hc.apache.org by gg...@apache.org on 2015/08/27 00:29:06 UTC

svn commit: r1698031 - in /httpcomponents/httpclient/trunk/httpmime/src/main/java/org/apache/http/entity/mime: Header.java content/AbstractContentBody.java

Author: ggregory
Date: Wed Aug 26 22:29:05 2015
New Revision: 1698031

URL: http://svn.apache.org/r1698031
Log:
Statement unnecessarily nested within else clause.

Modified:
    httpcomponents/httpclient/trunk/httpmime/src/main/java/org/apache/http/entity/mime/Header.java
    httpcomponents/httpclient/trunk/httpmime/src/main/java/org/apache/http/entity/mime/content/AbstractContentBody.java

Modified: httpcomponents/httpclient/trunk/httpmime/src/main/java/org/apache/http/entity/mime/Header.java
URL: http://svn.apache.org/viewvc/httpcomponents/httpclient/trunk/httpmime/src/main/java/org/apache/http/entity/mime/Header.java?rev=1698031&r1=1698030&r2=1698031&view=diff
==============================================================================
--- httpcomponents/httpclient/trunk/httpmime/src/main/java/org/apache/http/entity/mime/Header.java (original)
+++ httpcomponents/httpclient/trunk/httpmime/src/main/java/org/apache/http/entity/mime/Header.java Wed Aug 26 22:29:05 2015
@@ -88,9 +88,8 @@ public class Header implements Iterable<
         final List<MinimalField> list = this.fieldMap.get(key);
         if (list == null || list.isEmpty()) {
             return Collections.emptyList();
-        } else {
-            return new ArrayList<>(list);
         }
+        return new ArrayList<>(list);
     }
 
     public int removeFields(final String name) {

Modified: httpcomponents/httpclient/trunk/httpmime/src/main/java/org/apache/http/entity/mime/content/AbstractContentBody.java
URL: http://svn.apache.org/viewvc/httpcomponents/httpclient/trunk/httpmime/src/main/java/org/apache/http/entity/mime/content/AbstractContentBody.java?rev=1698031&r1=1698030&r2=1698031&view=diff
==============================================================================
--- httpcomponents/httpclient/trunk/httpmime/src/main/java/org/apache/http/entity/mime/content/AbstractContentBody.java (original)
+++ httpcomponents/httpclient/trunk/httpmime/src/main/java/org/apache/http/entity/mime/content/AbstractContentBody.java Wed Aug 26 22:29:05 2015
@@ -67,9 +67,8 @@ public abstract class AbstractContentBod
         final int i = mimeType.indexOf('/');
         if (i != -1) {
             return mimeType.substring(0, i);
-        } else {
-            return mimeType;
         }
+        return mimeType;
     }
 
     @Override
@@ -78,9 +77,8 @@ public abstract class AbstractContentBod
         final int i = mimeType.indexOf('/');
         if (i != -1) {
             return mimeType.substring(i + 1);
-        } else {
-            return null;
         }
+        return null;
     }
 
     @Override