You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm-commits@maven.apache.org by ca...@apache.org on 2006/03/07 08:07:51 UTC

svn commit: r383808 - /maven/scm/trunk/maven-scm-api/src/main/java/org/apache/maven/scm/command/add/AddScmResult.java

Author: carlos
Date: Mon Mar  6 23:07:50 2006
New Revision: 383808

URL: http://svn.apache.org/viewcvs?rev=383808&view=rev
Log:
Prevent null values in addedFiles

Modified:
    maven/scm/trunk/maven-scm-api/src/main/java/org/apache/maven/scm/command/add/AddScmResult.java

Modified: maven/scm/trunk/maven-scm-api/src/main/java/org/apache/maven/scm/command/add/AddScmResult.java
URL: http://svn.apache.org/viewcvs/maven/scm/trunk/maven-scm-api/src/main/java/org/apache/maven/scm/command/add/AddScmResult.java?rev=383808&r1=383807&r2=383808&view=diff
==============================================================================
--- maven/scm/trunk/maven-scm-api/src/main/java/org/apache/maven/scm/command/add/AddScmResult.java (original)
+++ maven/scm/trunk/maven-scm-api/src/main/java/org/apache/maven/scm/command/add/AddScmResult.java Mon Mar  6 23:07:50 2006
@@ -18,9 +18,12 @@
 
 import org.apache.maven.scm.ScmResult;
 
+import java.util.ArrayList;
 import java.util.List;
 
 /**
+ * Result of adding files to the SCM
+ * 
  * @author <a href="mailto:brett@apache.org">Brett Porter</a>
  * @version $Id$
  */
@@ -32,12 +35,19 @@
     public AddScmResult( String commandLine, String providerMessage, String commandOutput, boolean success )
     {
         super( commandLine, providerMessage, commandOutput, success );
+
+        addedFiles = new ArrayList( 0 );
     }
 
     public AddScmResult( String commandLine, List addedFiles )
     {
         super( commandLine, null, null, true );
 
+        if ( addedFiles == null )
+        {
+            throw new NullPointerException( "addedFiles can't be null" );
+        }
+
         this.addedFiles = addedFiles;
     }
 
@@ -45,9 +55,19 @@
     {
         super( result );
 
+        if ( addedFiles == null )
+        {
+            throw new NullPointerException( "addedFiles can't be null" );
+        }
+
         this.addedFiles = addedFiles;
     }
 
+    /**
+     * List with all the added files in the SCM operation
+     * 
+     * @return non null list of added files
+     */
     public List getAddedFiles()
     {
         return addedFiles;