[jbpm-commits] JBoss JBPM SVN: r2600 - jbpm3/trunk/modules/core/src/main/java/org/jbpm/instantiation.

[do-not-reply at jboss.org]

Author: camunda
Date: 2008-10-23 09:39:39 -0400 (Thu, 23 Oct 2008)
New Revision: 2600

Modified:
   jbpm3/trunk/modules/core/src/main/java/org/jbpm/instantiation/Delegation.java
Log:
added comments on selecting class loader for delegation

Modified: jbpm3/trunk/modules/core/src/main/java/org/jbpm/instantiation/Delegation.java
===================================================================
--- jbpm3/trunk/modules/core/src/main/java/org/jbpm/instantiation/Delegation.java	2008-10-23 12:18:48 UTC (rev 2599)
+++ jbpm3/trunk/modules/core/src/main/java/org/jbpm/instantiation/Delegation.java	2008-10-23 13:39:39 UTC (rev 2600)
@@ -132,7 +132,11 @@
 
     Object newInstance = null; 
 
-    // find the classloader to use
+    // The thread class loader was set before the instantiation correctly 
+    // to the ProcesClassLoader which can be directly used here
+    // If we would construct a JbpmConfiguration.getProcessClassLoder here
+    // we would have the hierarchy ProcessClassLoader -> ProcessClassLoader -> Context...
+    // this is one ProcessClassLoader too much
     ClassLoader classLoader = Thread.currentThread().getContextClassLoader();
     
     // load the class that needs to be instantiated