From e5f052d8d97930c7b0643ff4fb8bda9d7d16bbdf Mon Sep 17 00:00:00 2001 From: Prathamesh Hukkeri Date: Sun, 12 Jul 2026 21:07:09 +0530 Subject: [PATCH] fix: remove dead code catch block for ArtifactNotFoundException The ArtifactNotFoundException catch block was unreachable because ArtifactNotFoundException extends ArtifactResolutionException, which is caught first. This removes the dead code and the unused import. Closes #92 --- .../maven/shared/io/location/ArtifactLocatorStrategy.java | 5 ----- 1 file changed, 5 deletions(-) diff --git a/src/main/java/org/apache/maven/shared/io/location/ArtifactLocatorStrategy.java b/src/main/java/org/apache/maven/shared/io/location/ArtifactLocatorStrategy.java index ced388d..7273a51 100644 --- a/src/main/java/org/apache/maven/shared/io/location/ArtifactLocatorStrategy.java +++ b/src/main/java/org/apache/maven/shared/io/location/ArtifactLocatorStrategy.java @@ -23,7 +23,6 @@ import org.apache.maven.artifact.Artifact; import org.apache.maven.artifact.factory.ArtifactFactory; import org.apache.maven.artifact.repository.ArtifactRepository; -import org.apache.maven.artifact.resolver.ArtifactNotFoundException; import org.apache.maven.artifact.resolver.ArtifactResolutionException; import org.apache.maven.artifact.resolver.ArtifactResolver; import org.apache.maven.shared.io.logging.MessageHolder; @@ -162,10 +161,6 @@ public Location resolve(String locationSpecification, MessageHolder messageHolde messageHolder.addMessage( "Failed to resolve artifact: " + artifact.getId() + " for location: " + locationSpecification, e); - } catch (ArtifactNotFoundException e) { - messageHolder.addMessage( - "Failed to resolve artifact: " + artifact.getId() + " for location: " + locationSpecification, - e); } } else { messageHolder.addMessage("Invalid artifact specification: \'" + locationSpecification