You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by ol...@apache.org on 2011/08/27 15:33:19 UTC
svn commit: r1162348 -
/maven/maven-3/trunk/maven-core/src/main/java/org/apache/maven/execution/DefaultMavenExecutionResult.java
Author: olamy
Date: Sat Aug 27 13:33:19 2011
New Revision: 1162348
URL: http://svn.apache.org/viewvc?rev=1162348&view=rev
Log:
prevent ConcurrentModificationException
Modified:
maven/maven-3/trunk/maven-core/src/main/java/org/apache/maven/execution/DefaultMavenExecutionResult.java
Modified: maven/maven-3/trunk/maven-core/src/main/java/org/apache/maven/execution/DefaultMavenExecutionResult.java
URL: http://svn.apache.org/viewvc/maven/maven-3/trunk/maven-core/src/main/java/org/apache/maven/execution/DefaultMavenExecutionResult.java?rev=1162348&r1=1162347&r2=1162348&view=diff
==============================================================================
--- maven/maven-3/trunk/maven-core/src/main/java/org/apache/maven/execution/DefaultMavenExecutionResult.java (original)
+++ maven/maven-3/trunk/maven-core/src/main/java/org/apache/maven/execution/DefaultMavenExecutionResult.java Sat Aug 27 13:33:19 2011
@@ -24,6 +24,7 @@ import java.util.Collections;
import java.util.IdentityHashMap;
import java.util.List;
import java.util.Map;
+import java.util.concurrent.CopyOnWriteArrayList;
import org.apache.maven.project.DependencyResolutionResult;
import org.apache.maven.project.MavenProject;
@@ -38,7 +39,7 @@ public class DefaultMavenExecutionResult
private DependencyResolutionResult dependencyResolutionResult;
- private List<Throwable> exceptions;
+ private List<Throwable> exceptions = new CopyOnWriteArrayList<Throwable>( );
private Map<MavenProject, BuildSummary> buildSummaries;
@@ -85,11 +86,6 @@ public class DefaultMavenExecutionResult
public MavenExecutionResult addException( Throwable t )
{
- if ( exceptions == null )
- {
- exceptions = new ArrayList<Throwable>();
- }
-
exceptions.add( t );
return this;