svn commit: r1194941 - /commons/proper/bsf/trunk/src/main/java/org/apache/bsf/util/ReflectionUtils.java

[rony]

Author: rony
Date: Sat Oct 29 17:30:44 2011
New Revision: 1194941

URL: http://svn.apache.org/viewvc?rev=1194941&view=rev
Log:
Clean up a little bit (remove dead - commented - code)

Modified:
    
commons/proper/bsf/trunk/src/main/java/org/apache/bsf/util/ReflectionUtils.java

Modified: 
commons/proper/bsf/trunk/src/main/java/org/apache/bsf/util/ReflectionUtils.java
URL: 
http://svn.apache.org/viewvc/commons/proper/bsf/trunk/src/main/java/org/apache/bsf/util/ReflectionUtils.java?rev=1194941&r1=1194940&r2=1194941&view=diff
==============================================================================
--- 
commons/proper/bsf/trunk/src/main/java/org/apache/bsf/util/ReflectionUtils.java 
(original)
+++ 
commons/proper/bsf/trunk/src/main/java/org/apache/bsf/util/ReflectionUtils.java 
Sat Oct 29 17:30:44 2011
@@ -162,7 +162,6 @@ public class ReflectionUtils {
                   i++;
               }
 
-              // errMsg=errMsg+"}.";       // close set of event sets
               errMsg=errMsg+".";       // close set of event sets
           }
           throw new IllegalArgumentException (errMsg);
@@ -236,16 +235,9 @@ public class ReflectionUtils {
               IOException {
     if (argTypes != null) {
 
-/* original (remarked 20070917, rgf)
-      // find the right constructor and use that to create bean
-      Class cl = (cld != null) ? cld.loadClass (className)
-                   : Thread.currentThread().getContextClassLoader().loadClass 
(className); // rgf, 2006-01-05
-                                   // : Class.forName (className);
-*/
-            // TODO: final decision about class loading strategy
-            // rgf, 20070917: if class loader given, use that one, else try
-            //                the Thread's context class loader and then
-            //                the BSFMananger defining class loader
+            // if class loader given, use that one, else try
+            // the Thread's context class loader (if set) and then
+            // the BSFMananger defining class loader
           Class cl=null;
           ClassNotFoundException exCTX=null;
 
@@ -281,28 +273,6 @@ public class ReflectionUtils {
           }
 // -----------------------------
 
-/*
-          try {     // try supplied class loader
-              if (cld !=null)
-                  cl=cld.loadClass(className);
-          }
-          catch (ClassNotFoundException e01) {
-              try {
-                  
cl=Thread.currentThread().getContextClassLoader().loadClass(className);
-              }
-              catch (ClassNotFoundException e02)  {
-                  ClassLoader defCL=BSFManager.getDefinedClassLoader();
-                  if (cld != defCL) {
-                      cl=defCL.loadClass(className);
-                  }
-                  else
-                  {
-                      throw e01;        // re-throw original class not found 
exception
-                  }
-              }
-          }
-*/
-
       Constructor c = MethodUtils.getConstructor (cl, argTypes);
       return new Bean (cl, c.newInstance (args));
     } else {