You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by br...@apache.org on 2015/04/06 20:02:51 UTC
svn commit: r1671623 - /commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/ArrayUtils.java

Author: britter
Date: Mon Apr  6 18:02:50 2015
New Revision: 1671623

URL: http://svn.apache.org/r1671623
Log:
Simplify isEmpty be reusing getLength

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

Modified: commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/ArrayUtils.java
URL: http://svn.apache.org/viewvc/commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/ArrayUtils.java?rev=1671623&r1=1671622&r2=1671623&view=diff
==============================================================================
--- commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/ArrayUtils.java (original)
+++ commons/proper/lang/trunk/src/main/java/org/apache/commons/lang3/ArrayUtils.java Mon Apr  6 18:02:50 2015
@@ -3453,7 +3453,7 @@ public class ArrayUtils {
      * @since 2.1
      */
     public static boolean isEmpty(final Object[] array) {
-        return array == null || array.length == 0;
+        return getLength(array) == 0;
     }
 
     /**
@@ -3464,7 +3464,7 @@ public class ArrayUtils {
      * @since 2.1
      */
     public static boolean isEmpty(final long[] array) {
-        return array == null || array.length == 0;
+        return getLength(array) == 0;
     }
 
     /**
@@ -3475,7 +3475,7 @@ public class ArrayUtils {
      * @since 2.1
      */
     public static boolean isEmpty(final int[] array) {
-        return array == null || array.length == 0;
+        return getLength(array) == 0;
     }
 
     /**
@@ -3486,7 +3486,7 @@ public class ArrayUtils {
      * @since 2.1
      */
     public static boolean isEmpty(final short[] array) {
-        return array == null || array.length == 0;
+        return getLength(array) == 0;
     }
 
     /**
@@ -3497,7 +3497,7 @@ public class ArrayUtils {
      * @since 2.1
      */
     public static boolean isEmpty(final char[] array) {
-        return array == null || array.length == 0;
+        return getLength(array) == 0;
     }
 
     /**
@@ -3508,7 +3508,7 @@ public class ArrayUtils {
      * @since 2.1
      */
     public static boolean isEmpty(final byte[] array) {
-        return array == null || array.length == 0;
+        return getLength(array) == 0;
     }
 
     /**
@@ -3519,7 +3519,7 @@ public class ArrayUtils {
      * @since 2.1
      */
     public static boolean isEmpty(final double[] array) {
-        return array == null || array.length == 0;
+        return getLength(array) == 0;
     }
 
     /**
@@ -3530,7 +3530,7 @@ public class ArrayUtils {
      * @since 2.1
      */
     public static boolean isEmpty(final float[] array) {
-        return array == null || array.length == 0;
+        return getLength(array) == 0;
     }
 
     /**
@@ -3541,7 +3541,7 @@ public class ArrayUtils {
      * @since 2.1
      */
     public static boolean isEmpty(final boolean[] array) {
-        return array == null || array.length == 0;
+        return getLength(array) == 0;
     }
 
     // ----------------------------------------------------------------------