You are viewing a plain text version of this content. The canonical link for it is here.
Posted to ivy-commits@incubator.apache.org by gs...@apache.org on 2007/10/08 12:00:47 UTC
svn commit: r582792 -
/incubator/ivy/core/trunk/src/java/org/apache/ivy/core/resolve/ResolveEngine.java
Author: gscokart
Date: Mon Oct 8 05:00:46 2007
New Revision: 582792
URL: http://svn.apache.org/viewvc?rev=582792&view=rev
Log:
clean code formatting and add NPE messages for IVY-618
Modified:
incubator/ivy/core/trunk/src/java/org/apache/ivy/core/resolve/ResolveEngine.java
Modified: incubator/ivy/core/trunk/src/java/org/apache/ivy/core/resolve/ResolveEngine.java
URL: http://svn.apache.org/viewvc/incubator/ivy/core/trunk/src/java/org/apache/ivy/core/resolve/ResolveEngine.java?rev=582792&r1=582791&r2=582792&view=diff
==============================================================================
--- incubator/ivy/core/trunk/src/java/org/apache/ivy/core/resolve/ResolveEngine.java (original)
+++ incubator/ivy/core/trunk/src/java/org/apache/ivy/core/resolve/ResolveEngine.java Mon Oct 8 05:00:46 2007
@@ -98,7 +98,8 @@
* the sort engine to use to sort modules before producing the dependency resolution
* report. Must not be null.
*/
- public ResolveEngine(ResolveEngineSettings settings, EventManager eventManager, SortEngine sortEngine) {
+ public ResolveEngine(ResolveEngineSettings settings, EventManager eventManager,
+ SortEngine sortEngine) {
this.settings = settings;
this.eventManager = eventManager;
this.sortEngine = sortEngine;
@@ -194,8 +195,7 @@
}
/**
- * Resolve dependencies of a module described by a module descriptor Note: the method signature
- * is way too long, we should use a class to store the settings of the resolve.
+ * Resolve dependencies of a module described by a module descriptor.
*/
public ResolveReport resolve(ModuleDescriptor md, ResolveOptions options)
throws ParseException, IOException {
@@ -241,8 +241,8 @@
// revision id,
// for direct dependencies only.
// this is used by the deliver task to resolve dynamic revisions to static ones
- File ivyPropertiesInCache = cacheManager.getResolvedIvyPropertiesInCache(md
- .getResolvedModuleRevisionId());
+ File ivyPropertiesInCache = cacheManager.getResolvedIvyPropertiesInCache(
+ md.getResolvedModuleRevisionId());
Properties props = new Properties();
if (dependencies.length > 0) {
IvyNode root = dependencies[0].getRoot();
@@ -250,10 +250,24 @@
if (!dependencies[i].hasProblem()) {
DependencyDescriptor dd = dependencies[i].getDependencyDescriptor(root);
if (dd != null) {
- String rev = dependencies[i].getResolvedId().getRevision();
- String status = dependencies[i].getDescriptor().getStatus();
- props.put(dd.getDependencyRevisionId().encodeToString(), rev + " "
- + status);
+ ModuleRevisionId depResolvedId = dependencies[i].getResolvedId();
+ ModuleDescriptor depDescriptor = dependencies[i].getDescriptor();
+ ModuleRevisionId depRevisionId = dd.getDependencyRevisionId();
+ if (depResolvedId == null) {
+ throw new NullPointerException("getResolvedId() is null for "
+ + dependencies[i].toString());
+ }
+ if (depDescriptor == null) {
+ throw new NullPointerException("getDescriptor() is null for "
+ + dependencies[i].toString());
+ }
+ if (depRevisionId == null) {
+ throw new NullPointerException("getDependencyRevisionId() "
+ + "is null for " + dd.toString());
+ }
+ String rev = depResolvedId.getRevision();
+ String status = depDescriptor.getStatus();
+ props.put(depRevisionId.encodeToString(), rev + " " + status);
}
}
}