You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by be...@apache.org on 2009/07/20 20:32:53 UTC

svn commit: r795948 - /maven/components/trunk/maven-core/src/main/java/org/apache/maven/ReactorArtifactRepository.java

Author: bentmann
Date: Mon Jul 20 18:32:52 2009
New Revision: 795948

URL: http://svn.apache.org/viewvc?rev=795948&view=rev
Log:
o Optimized performance by utilizing that project identity is based on g:a:v which is just the map key

Modified:
    maven/components/trunk/maven-core/src/main/java/org/apache/maven/ReactorArtifactRepository.java

Modified: maven/components/trunk/maven-core/src/main/java/org/apache/maven/ReactorArtifactRepository.java
URL: http://svn.apache.org/viewvc/maven/components/trunk/maven-core/src/main/java/org/apache/maven/ReactorArtifactRepository.java?rev=795948&r1=795947&r2=795948&view=diff
==============================================================================
--- maven/components/trunk/maven-core/src/main/java/org/apache/maven/ReactorArtifactRepository.java (original)
+++ maven/components/trunk/maven-core/src/main/java/org/apache/maven/ReactorArtifactRepository.java Mon Jul 20 18:32:52 2009
@@ -20,9 +20,12 @@
 {
     private Map<String, MavenProject> reactorProjects;
 
+    private final int hashCode;
+
     public ReactorArtifactRepository( Map<String, MavenProject> reactorProjects )
     {
         this.reactorProjects = reactorProjects;
+        hashCode = ( reactorProjects != null ) ? reactorProjects.keySet().hashCode() : 0;
     }
 
     @Override
@@ -182,7 +185,7 @@
     @Override
     public int hashCode()
     {
-        return reactorProjects != null ? reactorProjects.hashCode() : 0;
+        return hashCode;
     }
 
     @Override
@@ -203,6 +206,6 @@
 
         ReactorArtifactRepository other = (ReactorArtifactRepository) obj;
 
-        return eq( reactorProjects, other.reactorProjects );
+        return eq( reactorProjects.keySet(), other.reactorProjects.keySet() );
     }
 }