Use for loops instead of streams

Author: gsmet

src/main/java/org/apache/maven/plugin/compiler/DeltaList.java

@@ -20,22 +20,29 @@
 
 import java.util.Collection;
 import java.util.Collections;
-import java.util.List;
-import java.util.stream.Collectors;
+import java.util.Set;
+import java.util.TreeSet;
 
 /**
  * Show the modifications between two lists.
  */
-final class DeltaList<E> {
+final class DeltaList<E extends Comparable<E>> {
 
-    private final List<E> added;
-    private final List<E> removed;
+    private final Set<E> added = new TreeSet<>();
+    private final Set<E> removed = new TreeSet<>();
     private final boolean hasChanged;
 
     DeltaList(Collection<E> oldList, Collection<E> newList) {
-        this.added = newList.stream().filter(i -> !oldList.contains(i)).sorted().collect(Collectors.toList());
-        this.removed =
-                oldList.stream().filter(i -> !newList.contains(i)).sorted().collect(Collectors.toList());
+        for (E newListItem : newList) {
+            if (!oldList.contains(newListItem)) {
+                added.add(newListItem);
+            }
+        }
+        for (E oldListItem : oldList) {
+            if (!newList.contains(oldListItem)) {
+                removed.add(oldListItem);
+            }
+        }
         this.hasChanged = !added.isEmpty() || !removed.isEmpty();
     }