aboutsummaryrefslogtreecommitdiff
path: root/src/main/java/org/apache/commons/lang3/builder
diff options
context:
space:
mode:
authorGary Gregory <garydgregory@gmail.com>2020-12-22 16:06:55 -0500
committerGary Gregory <garydgregory@gmail.com>2020-12-22 16:06:55 -0500
commit44b0bb5273db99db4a191b8b24f5676bd7a9c523 (patch)
tree794cb47446514b6e01762f0296060932f22a61f6 /src/main/java/org/apache/commons/lang3/builder
parent5aab41b3650db0fafeaf900a5b8c20342ea6726e (diff)
downloadapache-commons-lang-44b0bb5273db99db4a191b8b24f5676bd7a9c523.tar.gz
Use ArraySorter.
Diffstat (limited to 'src/main/java/org/apache/commons/lang3/builder')
-rw-r--r--src/main/java/org/apache/commons/lang3/builder/HashCodeBuilder.java5
-rw-r--r--src/main/java/org/apache/commons/lang3/builder/ReflectionToStringBuilder.java7
2 files changed, 5 insertions, 7 deletions
diff --git a/src/main/java/org/apache/commons/lang3/builder/HashCodeBuilder.java b/src/main/java/org/apache/commons/lang3/builder/HashCodeBuilder.java
index 0a95a74fb..75eb3e38b 100644
--- a/src/main/java/org/apache/commons/lang3/builder/HashCodeBuilder.java
+++ b/src/main/java/org/apache/commons/lang3/builder/HashCodeBuilder.java
@@ -20,12 +20,12 @@ package org.apache.commons.lang3.builder;
import java.lang.reflect.AccessibleObject;
import java.lang.reflect.Field;
import java.lang.reflect.Modifier;
-import java.util.Arrays;
import java.util.Collection;
import java.util.Comparator;
import java.util.HashSet;
import java.util.Set;
+import org.apache.commons.lang3.ArraySorter;
import org.apache.commons.lang3.ArrayUtils;
import org.apache.commons.lang3.Validate;
@@ -191,8 +191,7 @@ public class HashCodeBuilder implements Builder<Integer> {
try {
register(object);
// The elements in the returned array are not sorted and are not in any particular order.
- final Field[] fields = clazz.getDeclaredFields();
- Arrays.sort(fields, Comparator.comparing(Field::getName));
+ final Field[] fields = ArraySorter.sort(clazz.getDeclaredFields(), Comparator.comparing(Field::getName));
AccessibleObject.setAccessible(fields, true);
for (final Field field : fields) {
if (!ArrayUtils.contains(excludeFields, field.getName())
diff --git a/src/main/java/org/apache/commons/lang3/builder/ReflectionToStringBuilder.java b/src/main/java/org/apache/commons/lang3/builder/ReflectionToStringBuilder.java
index 66eb15cc2..63ad959d1 100644
--- a/src/main/java/org/apache/commons/lang3/builder/ReflectionToStringBuilder.java
+++ b/src/main/java/org/apache/commons/lang3/builder/ReflectionToStringBuilder.java
@@ -26,6 +26,7 @@ import java.util.Collection;
import java.util.Comparator;
import java.util.List;
+import org.apache.commons.lang3.ArraySorter;
import org.apache.commons.lang3.ArrayUtils;
import org.apache.commons.lang3.ClassUtils;
import org.apache.commons.lang3.Validate;
@@ -640,8 +641,7 @@ public class ReflectionToStringBuilder extends ToStringBuilder {
return;
}
// The elements in the returned array are not sorted and are not in any particular order.
- final Field[] fields = clazz.getDeclaredFields();
- Arrays.sort(fields, Comparator.comparing(Field::getName));
+ final Field[] fields = ArraySorter.sort(clazz.getDeclaredFields(), Comparator.comparing(Field::getName));
AccessibleObject.setAccessible(fields, true);
for (final Field field : fields) {
final String fieldName = field.getName();
@@ -801,8 +801,7 @@ public class ReflectionToStringBuilder extends ToStringBuilder {
this.excludeFieldNames = null;
} else {
//clone and remove nulls
- this.excludeFieldNames = toNoNullStringArray(excludeFieldNamesParam);
- Arrays.sort(this.excludeFieldNames);
+ this.excludeFieldNames = ArraySorter.sort(toNoNullStringArray(excludeFieldNamesParam));
}
return this;
}