This is an automated email from the ASF dual-hosted git repository.

sjaranowski pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/maven-dependency-plugin.git


The following commit(s) were added to refs/heads/master by this push:
     new ece9a38f Improve dependencies filtering in AbstractAnalyzeMojo
ece9a38f is described below

commit ece9a38fdc29ca84eb278ea7c4e38be7735af4af
Author: Slawomir Jaranowski <[email protected]>
AuthorDate: Sun Jan 18 11:27:04 2026 +0100

    Improve dependencies filtering in AbstractAnalyzeMojo
    
    For empty exclusion we cen return the same collection of artifacts
---
 .../maven/plugins/dependency/analyze/AbstractAnalyzeMojo.java     | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git 
a/src/main/java/org/apache/maven/plugins/dependency/analyze/AbstractAnalyzeMojo.java
 
b/src/main/java/org/apache/maven/plugins/dependency/analyze/AbstractAnalyzeMojo.java
index ac79d717..26ef281e 100644
--- 
a/src/main/java/org/apache/maven/plugins/dependency/analyze/AbstractAnalyzeMojo.java
+++ 
b/src/main/java/org/apache/maven/plugins/dependency/analyze/AbstractAnalyzeMojo.java
@@ -20,7 +20,6 @@ package org.apache.maven.plugins.dependency.analyze;
 
 import java.io.File;
 import java.io.StringWriter;
-import java.util.ArrayList;
 import java.util.Arrays;
 import java.util.Iterator;
 import java.util.LinkedHashMap;
@@ -558,9 +557,12 @@ public abstract class AbstractAnalyzeMojo extends 
AbstractMojo {
         }
     }
 
-    private List<Artifact> filterDependencies(Set<Artifact> artifacts, 
String[] excludes) {
+    private Set<Artifact> filterDependencies(Set<Artifact> artifacts, String[] 
excludes) {
+        if (excludes == null || excludes.length == 0) {
+            return artifacts;
+        }
         ArtifactFilter filter = new 
StrictPatternExcludesArtifactFilter(Arrays.asList(excludes));
-        List<Artifact> result = new ArrayList<>();
+        Set<Artifact> result = new LinkedHashSet<>();
 
         for (Iterator<Artifact> it = artifacts.iterator(); it.hasNext(); ) {
             Artifact artifact = it.next();

Reply via email to