diff --git a/src/main/java/org/apache/commons/lang3/ArrayUtils.java b/src/main/java/org/apache/commons/lang3/ArrayUtils.java index fc3eb4c3c..5aaeb17ff 100644 --- a/src/main/java/org/apache/commons/lang3/ArrayUtils.java +++ b/src/main/java/org/apache/commons/lang3/ArrayUtils.java @@ -8364,7 +8364,7 @@ public static boolean[] insert(final int index, final boolean[] array, final boo if (array == null) { return null; } - if (values == null || values.length == 0) { + if (ArrayUtils.isEmpty(values)) { return clone(array); } if (index < 0 || index > array.length) { @@ -8406,7 +8406,7 @@ public static byte[] insert(final int index, final byte[] array, final byte... v if (array == null) { return null; } - if (values == null || values.length == 0) { + if (ArrayUtils.isEmpty(values)) { return clone(array); } if (index < 0 || index > array.length) { @@ -8448,7 +8448,7 @@ public static char[] insert(final int index, final char[] array, final char... v if (array == null) { return null; } - if (values == null || values.length == 0) { + if (ArrayUtils.isEmpty(values)) { return clone(array); } if (index < 0 || index > array.length) { @@ -8490,7 +8490,7 @@ public static double[] insert(final int index, final double[] array, final doubl if (array == null) { return null; } - if (values == null || values.length == 0) { + if (ArrayUtils.isEmpty(values)) { return clone(array); } if (index < 0 || index > array.length) { @@ -8532,7 +8532,7 @@ public static float[] insert(final int index, final float[] array, final float.. if (array == null) { return null; } - if (values == null || values.length == 0) { + if (ArrayUtils.isEmpty(values)) { return clone(array); } if (index < 0 || index > array.length) { @@ -8574,7 +8574,7 @@ public static int[] insert(final int index, final int[] array, final int... valu if (array == null) { return null; } - if (values == null || values.length == 0) { + if (ArrayUtils.isEmpty(values)) { return clone(array); } if (index < 0 || index > array.length) { @@ -8616,7 +8616,7 @@ public static long[] insert(final int index, final long[] array, final long... v if (array == null) { return null; } - if (values == null || values.length == 0) { + if (ArrayUtils.isEmpty(values)) { return clone(array); } if (index < 0 || index > array.length) { @@ -8658,7 +8658,7 @@ public static short[] insert(final int index, final short[] array, final short.. if (array == null) { return null; } - if (values == null || values.length == 0) { + if (ArrayUtils.isEmpty(values)) { return clone(array); } if (index < 0 || index > array.length) { @@ -8710,7 +8710,7 @@ public static T[] insert(final int index, final T[] array, final T... values if (array == null) { return null; } - if (values == null || values.length == 0) { + if (ArrayUtils.isEmpty(values)) { return clone(array); } if (index < 0 || index > array.length) { diff --git a/src/main/java/org/apache/commons/lang3/text/StrBuilder.java b/src/main/java/org/apache/commons/lang3/text/StrBuilder.java index 262a9d772..435458802 100644 --- a/src/main/java/org/apache/commons/lang3/text/StrBuilder.java +++ b/src/main/java/org/apache/commons/lang3/text/StrBuilder.java @@ -1198,7 +1198,7 @@ public StrBuilder appendAll(@SuppressWarnings("unchecked") final T... array) * is fine, because it isn't inherited by subclasses, so each subclass must * vouch for itself whether its use of 'array' is safe. */ - if (array != null && array.length > 0) { + if (ArrayUtils.isNotEmpty(array)) { for (final Object element : array) { append(element); }