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"); }