You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by kh...@apache.org on 2014/12/31 11:28:31 UTC
maven git commit: [MNG-5702] Wrong reactor summary output while using
-T option Changing into a better solution which doesn't change the
implemented type of Map but use a synchronized one.
Repository: maven
Updated Branches:
refs/heads/master c407a4f7e -> cb91332d3
[MNG-5702] Wrong reactor summary output while using -T option
Changing into a better solution which doesn't change the
implemented type of Map but use a synchronized one.
Project: http://git-wip-us.apache.org/repos/asf/maven/repo
Commit: http://git-wip-us.apache.org/repos/asf/maven/commit/cb91332d
Tree: http://git-wip-us.apache.org/repos/asf/maven/tree/cb91332d
Diff: http://git-wip-us.apache.org/repos/asf/maven/diff/cb91332d
Branch: refs/heads/master
Commit: cb91332d3d5931bafdd5d717338f4a0351c61810
Parents: c407a4f
Author: Karl Heinz Marbaise <kh...@apache.org>
Authored: Wed Dec 31 11:25:55 2014 +0100
Committer: Karl Heinz Marbaise <kh...@apache.org>
Committed: Wed Dec 31 11:25:55 2014 +0100
----------------------------------------------------------------------
.../apache/maven/execution/DefaultMavenExecutionResult.java | 8 +++-----
1 file changed, 3 insertions(+), 5 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/maven/blob/cb91332d/maven-core/src/main/java/org/apache/maven/execution/DefaultMavenExecutionResult.java
----------------------------------------------------------------------
diff --git a/maven-core/src/main/java/org/apache/maven/execution/DefaultMavenExecutionResult.java b/maven-core/src/main/java/org/apache/maven/execution/DefaultMavenExecutionResult.java
index 55e87cd..1be812f 100644
--- a/maven-core/src/main/java/org/apache/maven/execution/DefaultMavenExecutionResult.java
+++ b/maven-core/src/main/java/org/apache/maven/execution/DefaultMavenExecutionResult.java
@@ -20,16 +20,14 @@ package org.apache.maven.execution;
*/
import java.util.Collections;
+import java.util.IdentityHashMap;
import java.util.List;
import java.util.Map;
-import java.util.concurrent.ConcurrentHashMap;
import java.util.concurrent.CopyOnWriteArrayList;
import org.apache.maven.project.DependencyResolutionResult;
import org.apache.maven.project.MavenProject;
-import com.google.common.collect.Maps;
-
/** @author Jason van Zyl */
public class DefaultMavenExecutionResult
implements MavenExecutionResult
@@ -42,7 +40,7 @@ public class DefaultMavenExecutionResult
private List<Throwable> exceptions = new CopyOnWriteArrayList<Throwable>();
- private Map<MavenProject, BuildSummary> buildSummaries = Maps.newConcurrentMap();
+ private Map<MavenProject, BuildSummary> buildSummaries = Collections.synchronizedMap(new IdentityHashMap<MavenProject, BuildSummary>());
public MavenExecutionResult setProject( MavenProject project )
{
@@ -107,7 +105,7 @@ public class DefaultMavenExecutionResult
{
if ( buildSummaries == null )
{
- buildSummaries = new ConcurrentHashMap<MavenProject, BuildSummary>();
+ buildSummaries = Collections.synchronizedMap(new IdentityHashMap<MavenProject, BuildSummary>());
}
buildSummaries.put( summary.getProject(), summary );
}