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-collections.git

commit f5e0f2c0994c6f25c23e1cb6acdd84dbe23c166b
Author: Gary Gregory <garydgreg...@gmail.com>
AuthorDate: Sun Jun 23 14:20:39 2024 -0400

    Use Collections.sort(List) for clarity
---
 .../java/org/apache/commons/collections4/CollectionUtilsTest.java     | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git 
a/src/test/java/org/apache/commons/collections4/CollectionUtilsTest.java 
b/src/test/java/org/apache/commons/collections4/CollectionUtilsTest.java
index 63fca4022..aaecfa991 100644
--- a/src/test/java/org/apache/commons/collections4/CollectionUtilsTest.java
+++ b/src/test/java/org/apache/commons/collections4/CollectionUtilsTest.java
@@ -596,7 +596,7 @@ public class CollectionUtilsTest extends MockTestCase {
 
         final List<Integer> combinedList = new ArrayList<>(collectionD);
         combinedList.addAll(collectionE);
-        combinedList.sort(null);
+        Collections.sort(combinedList);
 
         assertEquals(combinedList, result2, "Merge two lists 2");
 
@@ -640,7 +640,7 @@ public class CollectionUtilsTest extends MockTestCase {
         final Set<Integer> combinedSet = new HashSet<>(collectionD);
         combinedSet.addAll(collectionE);
         final List<Integer> combinedList = new ArrayList<>(combinedSet);
-        combinedList.sort(null);
+        Collections.sort(combinedList);
 
         assertEquals(combinedList, result2, "Merge two lists 2 - ignore 
duplicates");
     }

Reply via email to