[jboss-cvs] JBossAS SVN: r66595 - branches/JBoss_4_0_5_GA_CP/server/src/main/org/jboss/naming.

[jboss-cvs-commits at lists.jboss.org]

Author: vicky.kak at jboss.com
Date: 2007-10-31 03:23:35 -0400 (Wed, 31 Oct 2007)
New Revision: 66595

Modified:
   branches/JBoss_4_0_5_GA_CP/server/src/main/org/jboss/naming/ExternalContext.java
Log:
ASPATCH-299 changes.

Modified: branches/JBoss_4_0_5_GA_CP/server/src/main/org/jboss/naming/ExternalContext.java
===================================================================
--- branches/JBoss_4_0_5_GA_CP/server/src/main/org/jboss/naming/ExternalContext.java	2007-10-31 06:17:35 UTC (rev 66594)
+++ branches/JBoss_4_0_5_GA_CP/server/src/main/org/jboss/naming/ExternalContext.java	2007-10-31 07:23:35 UTC (rev 66595)
@@ -32,9 +32,10 @@
 import java.lang.reflect.Proxy;
 
 import java.net.URL;
-import java.util.ArrayList;
+import java.util.HashSet;
 import java.util.Hashtable;
 import java.util.Properties;
+import java.util.Set;
 
 import javax.management.MBeanServer;
 import javax.management.ObjectName;
@@ -52,7 +53,6 @@
 import javax.naming.spi.ObjectFactory;
 
 import org.jboss.system.ServiceMBeanSupport;
-import org.jboss.util.Classes;
 
 /**
  * A MBean that binds an arbitrary InitialContext into the JBoss default
@@ -521,13 +521,35 @@
          this.externalCtx = externalCtx;
       }
 
+      /**
+       * Returns an array containing all the unique interfaces implemented
+       * by the argument class c and all its superclasses. Interfaces that
+       * appear multiple times through inheritence are only accounted for once.
+       * 
+       * @param c - the class to start scanning for interfaces
+       */   
+      private static Class[] getAllUniqueInterfaces(Class c)
+      {
+         Set uniqueIfaces = new HashSet();
+         while (c != null )
+         {
+            Class[] ifaces = c.getInterfaces();
+            for (int n = 0; n < ifaces.length; n ++)
+            {
+               uniqueIfaces.add(ifaces[n]);
+            }
+            c = c.getSuperclass();
+         }
+         return (Class[])uniqueIfaces.toArray(new Class[uniqueIfaces.size()]);
+      }
+      
       static Context createProxyContext(Context ctx)
       {
          ClassLoader loader = Thread.currentThread().getContextClassLoader();
-         ArrayList ifaces = new ArrayList();
-         Classes.getAllInterfaces(ifaces, ctx.getClass());
-         Class[] interfaces = new Class[ifaces.size()];
-         ifaces.toArray(interfaces);
+         // TODO,JBAS-4753 - remove the above method and replace the following 
+         // line with a call to org.jboss.util.Classes.getAllUniqueInterfaces()
+         // when the next jboss-common release is made available.
+         Class[] interfaces = getAllUniqueInterfaces(ctx.getClass());
          InvocationHandler handler = new CachedContext(ctx);
          Context proxyCtx = (Context) Proxy.newProxyInstance(loader, interfaces, handler);
          return proxyCtx;