Commit c861f24b authored by Phillip Webb's avatar Phillip Webb

Switch mergeDependencies to use a set.

parent d36f9ce8
...@@ -19,14 +19,15 @@ package org.springframework.boot.dependency.tools; ...@@ -19,14 +19,15 @@ package org.springframework.boot.dependency.tools;
import java.util.ArrayList; import java.util.ArrayList;
import java.util.Iterator; import java.util.Iterator;
import java.util.LinkedHashMap; import java.util.LinkedHashMap;
import java.util.List; import java.util.LinkedHashSet;
import java.util.Map; import java.util.Map;
import java.util.Set;
import org.springframework.boot.dependency.tools.Dependency.Exclusion; import org.springframework.boot.dependency.tools.Dependency.Exclusion;
/** /**
* Abstract base implementation for {@link Dependencies}. * Abstract base implementation for {@link Dependencies}.
* *
* @author Phillip Webb * @author Phillip Webb
* @author Andy Wilkinson * @author Andy Wilkinson
* @since 1.1.0 * @since 1.1.0
...@@ -68,12 +69,11 @@ abstract class AbstractDependencies implements Dependencies { ...@@ -68,12 +69,11 @@ abstract class AbstractDependencies implements Dependencies {
private Dependency mergeDependencies(Dependency existingDependency, private Dependency mergeDependencies(Dependency existingDependency,
Dependency newDependency) { Dependency newDependency) {
List<Exclusion> combinedExclusions = new ArrayList<Exclusion>(); Set<Exclusion> combinedExclusions = new LinkedHashSet<Exclusion>();
combinedExclusions.addAll(existingDependency.getExclusions()); combinedExclusions.addAll(existingDependency.getExclusions());
combinedExclusions.addAll(newDependency.getExclusions()); combinedExclusions.addAll(newDependency.getExclusions());
return new Dependency(newDependency.getGroupId(), newDependency.getArtifactId(), return new Dependency(newDependency.getGroupId(), newDependency.getArtifactId(),
newDependency.getVersion(), combinedExclusions); newDependency.getVersion(), new ArrayList<Exclusion>(combinedExclusions));
} }
/** /**
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment