Collections.sort() can be replaced with List.sort()

Reports calls to Collections.sort(list, comparator) which could be replaced with list.sort(comparator).

Signed-off-by: Lars Grefer <eclipse@larsgrefer.de>

1 files changed, 3 insertions, 3 deletions

diff --git a/bcel-builder/src/main/java/org/aspectj/apache/bcel/generic/ClassGen.java b/bcel-builder/src/main/java/org/aspectj/apache/bcel/generic/ClassGen.java
index 943b77f6e..371013997 100644
--- a/bcel-builder/src/main/java/org/aspectj/apache/bcel/generic/ClassGen.java
+++ b/bcel-builder/src/main/java/org/aspectj/apache/bcel/generic/ClassGen.java
@@ -509,7 +509,7 @@ public class ClassGen extends Modifiers implements Cloneable {
 					relevantFields.add(field);
 				}
 			}
-			Collections.sort(relevantFields, new FieldComparator());
+			relevantFields.sort(new FieldComparator());
 			int relevantFlags = Constants.ACC_PUBLIC | Constants.ACC_PRIVATE | Constants.ACC_PROTECTED | Constants.ACC_STATIC
 					| Constants.ACC_FINAL | Constants.ACC_VOLATILE | Constants.ACC_TRANSIENT;
 			for (Field f : relevantFields) {
@@ -534,8 +534,8 @@ public class ClassGen extends Modifiers implements Cloneable {
 						relevantMethods.add(m);
 				}
 			}
-			Collections.sort(relevantCtors, new ConstructorComparator());
-			Collections.sort(relevantMethods, new MethodComparator());
+			relevantCtors.sort(new ConstructorComparator());
+			relevantMethods.sort(new MethodComparator());
 
 			// 5. If a class initializer exists, write out the following:
 			// 1. The name of the method, <clinit>.