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/06/27 14:28:05 UTC

svn commit: r1606056 - /commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/ClassUtils.java

Author: ggregory
Date: Fri Jun 27 12:28:05 2014
New Revision: 1606056

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

Modified:
    commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/ClassUtils.java

Modified: commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/ClassUtils.java
URL: http://svn.apache.org/viewvc/commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/ClassUtils.java?rev=1606056&r1=1606055&r2=1606056&view=diff
==============================================================================
--- commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/ClassUtils.java (original)
+++ commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/ClassUtils.java Fri Jun 27 12:28:05 2014
@@ -1117,24 +1117,23 @@ public class ClassUtils {
         }
         if (dim < 1) {
             return className;
+        }
+        if (className.startsWith("L")) {
+            className = className.substring(
+                1,
+                className.endsWith(";")
+                    ? className.length() - 1
+                    : className.length());
         } else {
-            if (className.startsWith("L")) {
-                className = className.substring(
-                    1,
-                    className.endsWith(";")
-                        ? className.length() - 1
-                        : className.length());
-            } else {
-                if (className.length() > 0) {
-                    className = reverseAbbreviationMap.get(className.substring(0, 1));
-                }
-            }
-            final StringBuilder canonicalClassNameBuffer = new StringBuilder(className);
-            for (int i = 0; i < dim; i++) {
-                canonicalClassNameBuffer.append("[]");
+            if (className.length() > 0) {
+                className = reverseAbbreviationMap.get(className.substring(0, 1));
             }
-            return canonicalClassNameBuffer.toString();
         }
+        final StringBuilder canonicalClassNameBuffer = new StringBuilder(className);
+        for (int i = 0; i < dim; i++) {
+            canonicalClassNameBuffer.append("[]");
+        }
+        return canonicalClassNameBuffer.toString();
     }
 
     /**