You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by rf...@apache.org on 2012/10/19 16:20:16 UTC

svn commit: r1400107 - /maven/plugins/trunk/maven-dependency-plugin/src/main/java/org/apache/maven/plugin/dependency/PurgeLocalRepositoryMojo.java

Author: rfscholte
Date: Fri Oct 19 14:20:16 2012
New Revision: 1400107

URL: http://svn.apache.org/viewvc?rev=1400107&view=rev
Log:
Small code-optimizations

Modified:
    maven/plugins/trunk/maven-dependency-plugin/src/main/java/org/apache/maven/plugin/dependency/PurgeLocalRepositoryMojo.java

Modified: maven/plugins/trunk/maven-dependency-plugin/src/main/java/org/apache/maven/plugin/dependency/PurgeLocalRepositoryMojo.java
URL: http://svn.apache.org/viewvc/maven/plugins/trunk/maven-dependency-plugin/src/main/java/org/apache/maven/plugin/dependency/PurgeLocalRepositoryMojo.java?rev=1400107&r1=1400106&r2=1400107&view=diff
==============================================================================
--- maven/plugins/trunk/maven-dependency-plugin/src/main/java/org/apache/maven/plugin/dependency/PurgeLocalRepositoryMojo.java (original)
+++ maven/plugins/trunk/maven-dependency-plugin/src/main/java/org/apache/maven/plugin/dependency/PurgeLocalRepositoryMojo.java Fri Oct 19 14:20:16 2012
@@ -39,6 +39,7 @@ import org.apache.maven.plugins.annotati
 import org.apache.maven.plugins.annotations.Parameter;
 import org.apache.maven.project.MavenProject;
 import org.codehaus.plexus.util.FileUtils;
+import org.codehaus.plexus.util.StringUtils;
 
 import java.io.File;
 import java.io.IOException;
@@ -264,12 +265,12 @@ public class PurgeLocalRepositoryMojo
     {
         for ( String pattern : inclusionPatterns )
         {
-            if ( pattern.equals( "" ) )
+            if ( StringUtils.isEmpty( pattern ) )
             {
                 throw new MojoExecutionException( "The groupId:artifactId for manualIncludes cannot be empty" );
             }
             String relativePath = gaStringtoPath( pattern );
-            File purgeDir = new File( localRepository.getBasedir() + "/" + relativePath );
+            File purgeDir = new File( localRepository.getBasedir(), relativePath );
             if ( purgeDir.exists() )
             {
                 try
@@ -336,7 +337,7 @@ public class PurgeLocalRepositoryMojo
 
         for ( Dependency dependency : dependencies )
         {
-            if ( dependency.getScope() != null && dependency.getScope().equals( Artifact.SCOPE_SYSTEM ) )
+            if ( Artifact.SCOPE_SYSTEM.equals( dependency.getScope() ))
             {
                 // Don't try to purge system dependencies
                 continue;