some clean up from inspections

git-svn-id: https://svn.apache.org/repos/asf/maven/components/trunk@163441 13f79535-47bb-0310-9956-ffa450edef68

maven-artifact/src/main/java/org/apache/maven/artifact/deployer/ArtifactDeployer.java

@@ -18,7 +18,6 @@
 
 import org.apache.maven.artifact.Artifact;
 import org.apache.maven.artifact.repository.ArtifactRepository;
-import org.apache.maven.artifact.repository.ArtifactRepository;
 
 import java.io.File;
 

maven-artifact/src/main/java/org/apache/maven/artifact/handler/JarHandler.java

@@ -1,9 +1,5 @@
 package org.apache.maven.artifact.handler;
 
-import org.apache.maven.artifact.Artifact;
-
-import java.io.File;
-
 /*
  * Copyright 2001-2004 The Apache Software Foundation.
  *

maven-artifact/src/main/java/org/apache/maven/artifact/handler/PluginHandler.java

@@ -16,10 +16,6 @@
  * limitations under the License.
  */
 
-import org.apache.maven.artifact.Artifact;
-
-import java.io.File;
-
 /**
  * @author <a href="mailto:jason@maven.org">Jason van Zyl</a>
  * @version $Id$

maven-artifact/src/main/java/org/apache/maven/artifact/handler/PomHandler.java

@@ -16,10 +16,6 @@
  * limitations under the License.
  */
 
-import org.apache.maven.artifact.Artifact;
-
-import java.io.File;
-
 /**
  * @author <a href="mailto:jason@maven.org">Jason van Zyl</a>
  * @version $Id$

maven-artifact/src/main/java/org/apache/maven/artifact/handler/TestHandler.java

@@ -16,10 +16,6 @@
  * limitations under the License.
  */
 
-import org.apache.maven.artifact.Artifact;
-
-import java.io.File;
-
 /**
  * @todo [BP] should be removed and replaced with scope if other design is approved
  * @author <a href="mailto:jason@maven.org">Jason van Zyl</a>

maven-artifact/src/main/java/org/apache/maven/artifact/installer/ArtifactInstaller.java

@@ -18,7 +18,6 @@
 
 import org.apache.maven.artifact.Artifact;
 import org.apache.maven.artifact.repository.ArtifactRepository;
-import org.apache.maven.artifact.repository.ArtifactRepository;
 
 import java.io.File;
 

maven-artifact/src/main/java/org/apache/maven/artifact/repository/ArtifactRepository.java

@@ -17,7 +17,6 @@
  */
 
 import org.apache.maven.wagon.repository.Repository;
-import org.codehaus.plexus.util.StringUtils;
 
 /**
  * This class is an abstraction of the location from/to resources

maven-artifact/src/main/java/org/apache/maven/artifact/request/ArtifactRequestTransformation.java

@@ -1,12 +1,5 @@
 package org.apache.maven.artifact.request;
 
-import org.apache.maven.artifact.Artifact;
-import org.apache.maven.artifact.repository.ArtifactRepository;
-import org.apache.maven.artifact.repository.ArtifactRepository;
-
-import java.util.Map;
-import java.util.Set;
-
 /*
  * Copyright 2001-2004 The Apache Software Foundation.
  *
@@ -23,6 +16,12 @@
  * limitations under the License.
  */
 
+import org.apache.maven.artifact.Artifact;
+import org.apache.maven.artifact.repository.ArtifactRepository;
+
+import java.util.Map;
+import java.util.Set;
+
 /**
  * @author <a href="mailto:jason@maven.org">Jason van Zyl</a>
  * @version $Id$

maven-artifact/src/main/java/org/apache/maven/artifact/resolver/DefaultArtifactResolver.java

@@ -62,7 +62,7 @@ public Artifact resolve( Artifact artifact,
         return artifact;
     }
 
-    private String LS = System.getProperty( "line.separator" );
+    private static final String LS = System.getProperty( "line.separator" );
 
     private String artifactNotFound( Artifact artifact, Set remoteRepositories )
     {

maven-artifact/src/test/java/org/apache/maven/artifact/installer/ArtifactInstallerTest.java

@@ -16,9 +16,8 @@
  * limitations under the License.
  */
 
-import org.apache.maven.artifact.ArtifactComponentTestCase;
 import org.apache.maven.artifact.Artifact;
-import org.apache.maven.artifact.DefaultArtifact;
+import org.apache.maven.artifact.ArtifactComponentTestCase;
 
 import java.io.File;
 

maven-artifact/src/test/java/org/apache/maven/artifact/manager/DefaultWagonManagerTest.java

@@ -35,30 +35,21 @@ public void testDefaultWagonManager()
 
         Wagon wagon = null;
 
-        try
+        wagon = (Wagon) wagonManager.getWagon( "a" );
-        {
-            wagon = (Wagon) wagonManager.getWagon( "a" );
 
-            assertNotNull( wagon );
+        assertNotNull( wagon );
 
-            wagon = (Wagon) wagonManager.getWagon( "b1" );
+        wagon = (Wagon) wagonManager.getWagon( "b1" );
 
-            assertNotNull( wagon );
+        assertNotNull( wagon );
 
-            wagon = (Wagon) wagonManager.getWagon( "b2" );
+        wagon = (Wagon) wagonManager.getWagon( "b2" );
 
-            assertNotNull( wagon );
+        assertNotNull( wagon );
 
-            wagon = (Wagon) wagonManager.getWagon( "c" );
+        wagon = (Wagon) wagonManager.getWagon( "c" );
 
-            assertNotNull( wagon );
+        assertNotNull( wagon );
-        }
-        catch ( Exception e )
-        {
-            e.printStackTrace();
-
-            fail( e.getMessage() );
-        }
 
         try
         {