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 )