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 2021/01/01 18:35:00 UTC
[commons-lang] branch master updated: CLean up odds and ends.
This is an automated email from the ASF dual-hosted git repository.
ggregory pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/commons-lang.git
The following commit(s) were added to refs/heads/master by this push:
new a36b59c CLean up odds and ends.
a36b59c is described below
commit a36b59c6f8ecc83ed2d931e59d871ecd4d9ff2b5
Author: Gary Gregory <ga...@gmail.com>
AuthorDate: Fri Jan 1 13:34:56 2021 -0500
CLean up odds and ends.
- Use EMPTY constant.
- Move a guard clause down to the implementating method from wrapper
method.
- Remove dead comments.
- Remove duplicate null check.
- Remove duplicate empty-string check.
- Use Charsets.toCharset).
---
.../java/org/apache/commons/lang3/ArrayUtils.java | 2 --
.../apache/commons/lang3/CharSequenceUtils.java | 5 ++-
.../java/org/apache/commons/lang3/StringUtils.java | 41 ++++++++--------------
3 files changed, 19 insertions(+), 29 deletions(-)
diff --git a/src/main/java/org/apache/commons/lang3/ArrayUtils.java b/src/main/java/org/apache/commons/lang3/ArrayUtils.java
index 889d80b..6c31302 100644
--- a/src/main/java/org/apache/commons/lang3/ArrayUtils.java
+++ b/src/main/java/org/apache/commons/lang3/ArrayUtils.java
@@ -6823,8 +6823,6 @@ public static int indexOf(final int[] array, final int valueToFind, int startInd
}
}
- // Reverse
- //-----------------------------------------------------------------------
/**
* <p>Reverses the order of the given array.
*
diff --git a/src/main/java/org/apache/commons/lang3/CharSequenceUtils.java b/src/main/java/org/apache/commons/lang3/CharSequenceUtils.java
index 17669ba..b08cb54 100644
--- a/src/main/java/org/apache/commons/lang3/CharSequenceUtils.java
+++ b/src/main/java/org/apache/commons/lang3/CharSequenceUtils.java
@@ -223,11 +223,14 @@ public class CharSequenceUtils {
* Used by the lastIndexOf(CharSequence methods) as a green implementation of lastIndexOf
*
* @param cs the {@code CharSequence} to be processed
- * @param searchChar the {@code CharSequence} to be searched for
+ * @param searchChar the {@code CharSequence} to find
* @param start the start index
* @return the index where the search sequence was found
*/
static int lastIndexOf(final CharSequence cs, final CharSequence searchChar, int start) {
+ if (searchChar == null || cs == null) {
+ return NOT_FOUND;
+ }
if (searchChar instanceof String) {
if (cs instanceof String) {
return ((String) cs).lastIndexOf((String) searchChar, start);
diff --git a/src/main/java/org/apache/commons/lang3/StringUtils.java b/src/main/java/org/apache/commons/lang3/StringUtils.java
index a35b002..49ffda4 100644
--- a/src/main/java/org/apache/commons/lang3/StringUtils.java
+++ b/src/main/java/org/apache/commons/lang3/StringUtils.java
@@ -1324,7 +1324,7 @@ public class StringUtils {
* @since 3.0 Changed signature from containsNone(String, String) to containsNone(CharSequence, String)
*/
public static boolean containsNone(final CharSequence cs, final String invalidChars) {
- if (cs == null || invalidChars == null) {
+ if (invalidChars == null) {
return true;
}
return containsNone(cs, invalidChars.toCharArray());
@@ -4857,7 +4857,7 @@ public class StringUtils {
final Iterator<Object> iterator = Arrays.asList(objects).iterator();
while (iterator.hasNext()) {
- final String value = Objects.toString(iterator.next(), "");
+ final String value = Objects.toString(iterator.next(), EMPTY);
result.append(value);
if (iterator.hasNext()) {
@@ -4892,7 +4892,7 @@ public class StringUtils {
* @since 3.0 Changed signature from lastIndexOf(String, String) to lastIndexOf(CharSequence, CharSequence)
*/
public static int lastIndexOf(final CharSequence seq, final CharSequence searchSeq) {
- if (seq == null || searchSeq == null) {
+ if (seq == null) {
return INDEX_NOT_FOUND;
}
return CharSequenceUtils.lastIndexOf(seq, searchSeq, seq.length());
@@ -4934,9 +4934,6 @@ public class StringUtils {
* @since 3.0 Changed signature from lastIndexOf(String, String, int) to lastIndexOf(CharSequence, CharSequence, int)
*/
public static int lastIndexOf(final CharSequence seq, final CharSequence searchSeq, final int startPos) {
- if (seq == null || searchSeq == null) {
- return INDEX_NOT_FOUND;
- }
return CharSequenceUtils.lastIndexOf(seq, searchSeq, startPos);
}
@@ -6103,9 +6100,6 @@ public class StringUtils {
* @since 3.5
*/
public static String removeIgnoreCase(final String str, final String remove) {
- if (isEmpty(str) || isEmpty(remove)) {
- return str;
- }
return replaceIgnoreCase(str, remove, EMPTY, -1);
}
@@ -6206,11 +6200,8 @@ public class StringUtils {
* @since 2.4
*/
public static String removeStartIgnoreCase(final String str, final String remove) {
- if (isEmpty(str) || isEmpty(remove)) {
- return str;
- }
- if (startsWithIgnoreCase(str, remove)) {
- return str.substring(remove.length());
+ if (str != null && startsWithIgnoreCase(str, remove)) {
+ return str.substring(length(remove));
}
return str;
}
@@ -7740,7 +7731,6 @@ public class StringUtils {
return substrings.toArray(ArrayUtils.EMPTY_STRING_ARRAY);
}
- // -----------------------------------------------------------------------
/**
* <p>Splits the provided text into an array, using whitespace as the
* separator, preserving all tokens, including empty tokens created by
@@ -8070,10 +8060,12 @@ public class StringUtils {
if (str == null || prefix == null) {
return str == prefix;
}
- if (prefix.length() > str.length()) {
+ // Get length once instead of twice in the unlikely case that it changes.
+ final int preLen = prefix.length();
+ if (preLen > str.length()) {
return false;
}
- return CharSequenceUtils.regionMatches(str, ignoreCase, 0, prefix, 0, prefix.length());
+ return CharSequenceUtils.regionMatches(str, ignoreCase, 0, prefix, 0, preLen);
}
/**
@@ -8189,9 +8181,6 @@ public class StringUtils {
* @return the stripped String, {@code null} if null String input
*/
public static String strip(String str, final String stripChars) {
- if (isEmpty(str)) {
- return str;
- }
str = stripStart(str, stripChars);
return stripEnd(str, stripChars);
}
@@ -8961,19 +8950,19 @@ public class StringUtils {
* StringUtils.toCodePoints("") = [] // empty array
* </pre>
*
- * @param str the character sequence to convert
+ * @param cs the character sequence to convert
* @return an array of code points
* @since 3.6
*/
- public static int[] toCodePoints(final CharSequence str) {
- if (str == null) {
+ public static int[] toCodePoints(final CharSequence cs) {
+ if (cs == null) {
return null;
}
- if (str.length() == 0) {
+ if (cs.length() == 0) {
return ArrayUtils.EMPTY_INT_ARRAY;
}
- final String s = str.toString();
+ final String s = cs.toString();
final int[] result = new int[s.codePointCount(0, s.length())];
int index = 0;
for (int i = 0; i < result.length; i++) {
@@ -9039,7 +9028,7 @@ public class StringUtils {
*/
@Deprecated
public static String toString(final byte[] bytes, final String charsetName) throws UnsupportedEncodingException {
- return charsetName != null ? new String(bytes, charsetName) : new String(bytes, Charset.defaultCharset());
+ return new String(bytes, Charsets.toCharset(charsetName));
}
/**