You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by hb...@apache.org on 2012/02/14 22:55:37 UTC

svn commit: r1244229 - in /maven/maven-3/trunk/maven-core/src/main/java/org/apache/maven: project/ project/artifact/ toolchain/

Author: hboutemy
Date: Tue Feb 14 21:55:36 2012
New Revision: 1244229

URL: http://svn.apache.org/viewvc?rev=1244229&view=rev
Log:
generics and Java 5 constructs

Modified:
    maven/maven-3/trunk/maven-core/src/main/java/org/apache/maven/project/DefaultMavenProjectHelper.java
    maven/maven-3/trunk/maven-core/src/main/java/org/apache/maven/project/MavenProjectHelper.java
    maven/maven-3/trunk/maven-core/src/main/java/org/apache/maven/project/artifact/AttachedArtifact.java
    maven/maven-3/trunk/maven-core/src/main/java/org/apache/maven/project/artifact/MavenMetadataSource.java
    maven/maven-3/trunk/maven-core/src/main/java/org/apache/maven/toolchain/DefaultToolchain.java
    maven/maven-3/trunk/maven-core/src/main/java/org/apache/maven/toolchain/ToolchainPrivate.java

Modified: maven/maven-3/trunk/maven-core/src/main/java/org/apache/maven/project/DefaultMavenProjectHelper.java
URL: http://svn.apache.org/viewvc/maven/maven-3/trunk/maven-core/src/main/java/org/apache/maven/project/DefaultMavenProjectHelper.java?rev=1244229&r1=1244228&r2=1244229&view=diff
==============================================================================
--- maven/maven-3/trunk/maven-core/src/main/java/org/apache/maven/project/DefaultMavenProjectHelper.java (original)
+++ maven/maven-3/trunk/maven-core/src/main/java/org/apache/maven/project/DefaultMavenProjectHelper.java Tue Feb 14 21:55:36 2012
@@ -106,7 +106,7 @@ public class DefaultMavenProjectHelper
         }
     }
 
-    public void addResource( MavenProject project, String resourceDirectory, List includes, List excludes )
+    public void addResource( MavenProject project, String resourceDirectory, List<String> includes, List<String> excludes )
     {
         Resource resource = new Resource();
         resource.setDirectory( resourceDirectory );
@@ -116,7 +116,7 @@ public class DefaultMavenProjectHelper
         project.addResource( resource );
     }
 
-    public void addTestResource( MavenProject project, String resourceDirectory, List includes, List excludes )
+    public void addTestResource( MavenProject project, String resourceDirectory, List<String> includes, List<String> excludes )
     {
         Resource resource = new Resource();
         resource.setDirectory( resourceDirectory );

Modified: maven/maven-3/trunk/maven-core/src/main/java/org/apache/maven/project/MavenProjectHelper.java
URL: http://svn.apache.org/viewvc/maven/maven-3/trunk/maven-core/src/main/java/org/apache/maven/project/MavenProjectHelper.java?rev=1244229&r1=1244228&r2=1244229&view=diff
==============================================================================
--- maven/maven-3/trunk/maven-core/src/main/java/org/apache/maven/project/MavenProjectHelper.java (original)
+++ maven/maven-3/trunk/maven-core/src/main/java/org/apache/maven/project/MavenProjectHelper.java Tue Feb 14 21:55:36 2012
@@ -32,8 +32,8 @@ public interface MavenProjectHelper
 
     void attachArtifact( MavenProject project, String artifactType, String artifactClassifier, File artifactFile );
 
-    void addResource( MavenProject project, String resourceDirectory, List includes, List excludes );
+    void addResource( MavenProject project, String resourceDirectory, List<String> includes, List<String> excludes );
 
-    void addTestResource( MavenProject project, String resourceDirectory, List includes, List excludes );
+    void addTestResource( MavenProject project, String resourceDirectory, List<String> includes, List<String> excludes );
 
 }

Modified: maven/maven-3/trunk/maven-core/src/main/java/org/apache/maven/project/artifact/AttachedArtifact.java
URL: http://svn.apache.org/viewvc/maven/maven-3/trunk/maven-core/src/main/java/org/apache/maven/project/artifact/AttachedArtifact.java?rev=1244229&r1=1244228&r2=1244229&view=diff
==============================================================================
--- maven/maven-3/trunk/maven-core/src/main/java/org/apache/maven/project/artifact/AttachedArtifact.java (original)
+++ maven/maven-3/trunk/maven-core/src/main/java/org/apache/maven/project/artifact/AttachedArtifact.java Tue Feb 14 21:55:36 2012
@@ -25,6 +25,7 @@ import org.apache.maven.artifact.Invalid
 import org.apache.maven.artifact.handler.ArtifactHandler;
 import org.apache.maven.artifact.metadata.ArtifactMetadata;
 import org.apache.maven.artifact.repository.ArtifactRepository;
+import org.apache.maven.artifact.versioning.ArtifactVersion;
 import org.apache.maven.artifact.versioning.VersionRange;
 
 import java.util.Collection;
@@ -71,12 +72,12 @@ public class AttachedArtifact
             + " It is derived from the main artifact." );
     }
 
-    public List getAvailableVersions()
+    public List<ArtifactVersion> getAvailableVersions()
     {
         return parent.getAvailableVersions();
     }
 
-    public void setAvailableVersions( List availableVersions )
+    public void setAvailableVersions( List<ArtifactVersion> availableVersions )
     {
         throw new UnsupportedOperationException( "Cannot change the version information for an attached artifact."
             + " It is derived from the main artifact." );
@@ -176,9 +177,9 @@ public class AttachedArtifact
         // we must fail silently here to avoid problems with the artifact transformers.
     }
 
-    public Collection getMetadataList()
+    public Collection<ArtifactMetadata> getMetadataList()
     {
-        return Collections.EMPTY_LIST;
+        return Collections.emptyList();
     }
 
 }

Modified: maven/maven-3/trunk/maven-core/src/main/java/org/apache/maven/project/artifact/MavenMetadataSource.java
URL: http://svn.apache.org/viewvc/maven/maven-3/trunk/maven-core/src/main/java/org/apache/maven/project/artifact/MavenMetadataSource.java?rev=1244229&r1=1244228&r2=1244229&view=diff
==============================================================================
--- maven/maven-3/trunk/maven-core/src/main/java/org/apache/maven/project/artifact/MavenMetadataSource.java (original)
+++ maven/maven-3/trunk/maven-core/src/main/java/org/apache/maven/project/artifact/MavenMetadataSource.java Tue Feb 14 21:55:36 2012
@@ -52,7 +52,6 @@ import org.apache.maven.artifact.version
 import org.apache.maven.artifact.versioning.DefaultArtifactVersion;
 import org.apache.maven.artifact.versioning.InvalidVersionSpecificationException;
 import org.apache.maven.artifact.versioning.VersionRange;
-import org.apache.maven.execution.MavenSession;
 import org.apache.maven.model.Dependency;
 import org.apache.maven.model.DependencyManagement;
 import org.apache.maven.model.DistributionManagement;

Modified: maven/maven-3/trunk/maven-core/src/main/java/org/apache/maven/toolchain/DefaultToolchain.java
URL: http://svn.apache.org/viewvc/maven/maven-3/trunk/maven-core/src/main/java/org/apache/maven/toolchain/DefaultToolchain.java?rev=1244229&r1=1244228&r2=1244229&view=diff
==============================================================================
--- maven/maven-3/trunk/maven-core/src/main/java/org/apache/maven/toolchain/DefaultToolchain.java (original)
+++ maven/maven-3/trunk/maven-core/src/main/java/org/apache/maven/toolchain/DefaultToolchain.java Tue Feb 14 21:55:36 2012
@@ -20,7 +20,6 @@ package org.apache.maven.toolchain;
  */
 
 import java.util.HashMap;
-import java.util.Iterator;
 import java.util.Map;
 
 import org.apache.maven.toolchain.model.ToolchainModel;
@@ -74,12 +73,11 @@ public abstract class DefaultToolchain
     }
 
 
-    public boolean matchesRequirements( Map requirements )
+    public boolean matchesRequirements( Map<String, String> requirements )
     {
-        Iterator it = requirements.keySet().iterator();
-        while ( it.hasNext() )
+        for ( Map.Entry<String, String> requirement : requirements.entrySet() )
         {
-            String key = (String) it.next();
+            String key = requirement.getKey();
 
             RequirementMatcher matcher = provides.get( key );
 
@@ -88,7 +86,7 @@ public abstract class DefaultToolchain
                 getLog().debug( "Toolchain " + this + " is missing required property: " + key );
                 return false;
             }
-            if ( !matcher.matches( (String) requirements.get( key ) ) )
+            if ( !matcher.matches( requirement.getValue() ) )
             {
                 getLog().debug( "Toolchain " + this + " doesn't match required property: " + key );
                 return false;

Modified: maven/maven-3/trunk/maven-core/src/main/java/org/apache/maven/toolchain/ToolchainPrivate.java
URL: http://svn.apache.org/viewvc/maven/maven-3/trunk/maven-core/src/main/java/org/apache/maven/toolchain/ToolchainPrivate.java?rev=1244229&r1=1244228&r2=1244229&view=diff
==============================================================================
--- maven/maven-3/trunk/maven-core/src/main/java/org/apache/maven/toolchain/ToolchainPrivate.java (original)
+++ maven/maven-3/trunk/maven-core/src/main/java/org/apache/maven/toolchain/ToolchainPrivate.java Tue Feb 14 21:55:36 2012
@@ -37,7 +37,7 @@ public interface ToolchainPrivate
      * @param requirements Map<String, String> key value pair
      * @return
      */
-    boolean matchesRequirements( Map requirements );
+    boolean matchesRequirements( Map<String, String> requirements );
 
     ToolchainModel getModel();