(maven-assembly-plugin) branch master updated: Remove commented code (#1273)

[elharo]

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

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


The following commit(s) were added to refs/heads/master by this push:
     new 31ac192d Remove commented code (#1273)
31ac192d is described below

commit 31ac192dbee96da6afd81aaed434e33703dff9ce
Author: Elliotte Rusty Harold <elh...@users.noreply.github.com>
AuthorDate: Sun Nov 23 10:03:53 2025 -0600

    Remove commented code (#1273)
    
    * Remove commented code
    
    * spotless
---
 .../maven/plugins/assembly/utils/FilterUtils.java  |  9 -------
 .../plugins/assembly/utils/ProjectUtilsTest.java   | 29 ----------------------
 2 files changed, 38 deletions(-)

diff --git 
a/src/main/java/org/apache/maven/plugins/assembly/utils/FilterUtils.java 
b/src/main/java/org/apache/maven/plugins/assembly/utils/FilterUtils.java
index b0f54cf4..99b7e2c5 100644
--- a/src/main/java/org/apache/maven/plugins/assembly/utils/FilterUtils.java
+++ b/src/main/java/org/apache/maven/plugins/assembly/utils/FilterUtils.java
@@ -39,9 +39,6 @@ import 
org.apache.maven.shared.artifact.filter.StatisticsReportingArtifactFilter
 import org.apache.maven.shared.artifact.filter.resolve.ScopeFilter;
 import org.slf4j.Logger;
 
-/**
- *
- */
 public final class FilterUtils {
 
     private FilterUtils() {}
@@ -125,12 +122,6 @@ public final class FilterUtils {
             allFilters.add(excludeFilter);
         }
 
-        // FIXME: Why is this added twice??
-        // if ( additionalFilters != null && !additionalFilters.isEmpty() )
-        // {
-        // allFilters.addAll( additionalFilters );
-        // }
-
         for (final Iterator<Artifact> it = artifacts.iterator(); it.hasNext(); 
) {
             final Artifact artifact = it.next();
 
diff --git 
a/src/test/java/org/apache/maven/plugins/assembly/utils/ProjectUtilsTest.java 
b/src/test/java/org/apache/maven/plugins/assembly/utils/ProjectUtilsTest.java
index 7633eea8..4bdd6e54 100644
--- 
a/src/test/java/org/apache/maven/plugins/assembly/utils/ProjectUtilsTest.java
+++ 
b/src/test/java/org/apache/maven/plugins/assembly/utils/ProjectUtilsTest.java
@@ -208,33 +208,4 @@ public class ProjectUtilsTest {
             fail("Found unexpected projects in result: " + unexpected);
         }
     }
-
-    // private void verifyProjectsNotPresent( List verify, Set result )
-    // {
-    // List verifyCopy = new ArrayList( verify );
-    //
-    // for ( Iterator it = result.iterator(); it.hasNext(); )
-    // {
-    // MavenProject project = (MavenProject) it.next();
-    //
-    // for ( Iterator verifyIterator = verifyCopy.iterator(); 
verifyIterator.hasNext(); )
-    // {
-    // MavenProject verification = (MavenProject) verifyIterator.next();
-    //
-    // if ( verification.getId().equals( project.getId() ) )
-    // {
-    // verifyIterator.remove();
-    // }
-    // }
-    // }
-    //
-    // if ( verifyCopy.size() != verify.size() )
-    // {
-    // List found = new ArrayList( verify );
-    // found.removeAll( verifyCopy );
-    //
-    // fail( "Failed to verify absence of: " + found );
-    // }
-    // }
-
 }