Cosmetic changes to ProxyServices SPI; better comments

weld · Jul 13, 2010 · c0bd09b · c0bd09b
1 parent 2949053
commit c0bd09b
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 10 deletions.
diff --git a/impl/src/main/java/org/jboss/weld/bean/proxy/util/SerializableProxy.java b/impl/src/main/java/org/jboss/weld/bean/proxy/util/SerializableProxy.java
@@ -26,7 +26,6 @@
 import java.io.ObjectStreamException;
 import java.io.Serializable;
 import java.lang.reflect.Type;
-import java.util.Arrays;
 import java.util.Set;
 
 import org.jboss.weld.Container;
@@ -103,7 +102,7 @@ private void writeObject(ObjectOutputStream out) throws IOException
    private void readObject(ObjectInputStream in) throws IOException, ClassNotFoundException
    {
       in.defaultReadObject();
-      Class<?> proxyBeanType = Container.instance().services().get(ProxyServices.class).loadProxySuperClass(proxySuperClassName);
+      Class<?> proxyBeanType = Container.instance().services().get(ProxyServices.class).loadBeanClass(proxySuperClassName);
       ArraySet<Type> proxyBeanInterfaces = loadInterfaces(); 
       Class<?> proxyClass = null;
       if (proxyClassName.endsWith(ProxyFactory.PROXY_SUFFIX))
@@ -131,7 +130,7 @@ private ArraySet<Type> loadInterfaces() throws ClassNotFoundException
       ArraySet<Type> interfaceClasses = new ArraySet<Type>(proxyInterfaces.size());
       for (String interfaceName : proxyInterfaces)
       {
-         interfaceClasses.add(proxyServices.loadProxySuperClass(interfaceName));
+         interfaceClasses.add(proxyServices.loadBeanClass(interfaceName));
       }
       return interfaceClasses;
    }

diff --git a/impl/src/main/java/org/jboss/weld/bean/proxy/util/SimpleProxyServices.java b/impl/src/main/java/org/jboss/weld/bean/proxy/util/SimpleProxyServices.java
@@ -38,7 +38,7 @@
 public class SimpleProxyServices implements ProxyServices
 {
 
-   public ClassLoader getClassLoader(final Class<?> type)
+   public ClassLoader getClassLoader(final Class<?> proxiedBeanType)
    {
       SecurityManager sm = System.getSecurityManager();
       if (sm != null)
@@ -47,26 +47,26 @@ public ClassLoader getClassLoader(final Class<?> type)
          {
             public ClassLoader run()
             {
-               return _getClassLoader(type);
+               return _getClassLoader(proxiedBeanType);
             }
          });
       }
       else
       {
-         return _getClassLoader(type);
+         return _getClassLoader(proxiedBeanType);
       }      
    }
 
-   private ClassLoader _getClassLoader(Class<?> type)
+   private ClassLoader _getClassLoader(Class<?> proxiedBeanType)
    {
       // return Thread.currentThread().getContextClassLoader();
-      if (type.getName().startsWith("java"))
+      if (proxiedBeanType.getName().startsWith("java"))
       {
          return this.getClass().getClassLoader();
       }
       else
       {
-         return type.getClassLoader();
+         return proxiedBeanType.getClassLoader();
       }
    }
 
@@ -94,7 +94,7 @@ public Object wrapForSerialization(Object proxyObject)
       return new SerializableProxy(proxyObject);
    }
 
-   public Class<?> loadProxySuperClass(final String className)
+   public Class<?> loadBeanClass(final String className)
    {
       try
       {