svn commit: r588052 - /maven/artifact/trunk/src/main/java/org/apache/maven/artifact/resolver/ArtifactResolutionResult.java

[ca...@apache.org]

Author: carlos
Date: Wed Oct 24 15:06:44 2007
New Revision: 588052

URL: http://svn.apache.org/viewvc?rev=588052&view=rev
Log:
Do not return null collections, makes life easier for clients

Modified:
    maven/artifact/trunk/src/main/java/org/apache/maven/artifact/resolver/ArtifactResolutionResult.java

Modified: maven/artifact/trunk/src/main/java/org/apache/maven/artifact/resolver/ArtifactResolutionResult.java
URL: http://svn.apache.org/viewvc/maven/artifact/trunk/src/main/java/org/apache/maven/artifact/resolver/ArtifactResolutionResult.java?rev=588052&r1=588051&r2=588052&view=diff
==============================================================================
--- maven/artifact/trunk/src/main/java/org/apache/maven/artifact/resolver/ArtifactResolutionResult.java (original)
+++ maven/artifact/trunk/src/main/java/org/apache/maven/artifact/resolver/ArtifactResolutionResult.java Wed Oct 24 15:06:44 2007
@@ -23,6 +23,7 @@
 import org.apache.maven.artifact.versioning.OverConstrainedVersionException;
 
 import java.util.ArrayList;
+import java.util.Collections;
 import java.util.HashSet;
 import java.util.Iterator;
 import java.util.List;
@@ -99,7 +100,7 @@
 
     public Set getArtifactResolutionNodes()
     {
-        return resolutionNodes;
+        return resolutionNodes == null ? Collections.EMPTY_SET : resolutionNodes;
     }
 
     public ArtifactResolutionResult setArtifactResolutionNodes( Set resolutionNodes )
@@ -114,7 +115,7 @@
 
     public List getMissingArtifacts()
     {
-        return missingArtifacts;
+        return missingArtifacts == null ? Collections.EMPTY_LIST : missingArtifacts;
     }
 
     public ArtifactResolutionResult addMissingArtifact( Artifact artifact )
@@ -163,7 +164,7 @@
 
     public List getVersionRangeViolations()
     {
-        return versionRangeViolations;
+        return versionRangeViolations == null ? Collections.EMPTY_LIST : versionRangeViolations;
     }
 
     // ------------------------------------------------------------------------
@@ -191,7 +192,7 @@
 
     public List getMetadataResolutionExceptions()
     {
-        return metadataResolutionExceptions;
+        return metadataResolutionExceptions == null ? Collections.EMPTY_LIST : metadataResolutionExceptions;
     }
 
     // ------------------------------------------------------------------------
@@ -214,7 +215,7 @@
 
     public List getErrorArtifactExceptions()
     {
-        return errorArtifactExceptions;
+        return errorArtifactExceptions == null ? Collections.EMPTY_LIST : errorArtifactExceptions;
     }
 
     // ------------------------------------------------------------------------
@@ -242,7 +243,7 @@
 
     public List getCircularDependencyExceptions()
     {
-        return circularDependencyExceptions;
+        return circularDependencyExceptions == null ? Collections.EMPTY_LIST : circularDependencyExceptions;
     }
 
     // ------------------------------------------------------------------------
@@ -253,7 +254,7 @@
 
     public List getRepositories()
     {
-        return repositories;
+        return repositories == null ? Collections.EMPTY_LIST : repositories;
     }
 
     public ArtifactResolutionResult setRepositories( List repositories )