[jboss-cvs] JBossAS SVN: r84374 - in projects/aop/trunk: aop/src/main/java/org/jboss/aop/reflectprototype and 10 other directories.

jboss-cvs-commits at lists.jboss.org jboss-cvs-commits at lists.jboss.org
Wed Feb 18 06:18:18 EST 2009


Author: kabir.khan at jboss.com
Date: 2009-02-18 06:18:18 -0500 (Wed, 18 Feb 2009)
New Revision: 84374

Modified:
   projects/aop/trunk/aop/src/main/java/org/jboss/aop/classpool/AOPClassPool.java
   projects/aop/trunk/aop/src/main/java/org/jboss/aop/reflectprototype/JavassistAopTypeInfoFactoryImpl.java
   projects/aop/trunk/aop/src/test/java/org/jboss/test/aop/standalone/PackageTestCase.java
   projects/aop/trunk/asintegration-core/src/test/java/org/jboss/test/aop/classpool/test/SimpleDelegatingClassPoolTestCase.java
   projects/aop/trunk/asintegration-jmx/src/main/java/org/jboss/aop/classpool/ucl/JBossUclDelegatingClassPoolFactory.java
   projects/aop/trunk/asintegration-jmx/src/test/java/org/jboss/test/aop/classpool/ucl/test/UclDelegatingClassPoolTestCase.java
   projects/aop/trunk/asintegration-mc/src/main/java/org/jboss/aop/asintegration/jboss5/JBoss5Integration.java
   projects/aop/trunk/asintegration-mc/src/main/java/org/jboss/aop/classpool/jbosscl/JBossClClassPoolDomain.java
   projects/aop/trunk/asintegration-mc/src/main/java/org/jboss/aop/classpool/jbosscl/JBossClDelegatingClassPoolFactory.java
   projects/aop/trunk/asintegration-mc/src/test/java/org/jboss/test/aop/classpool/jbosscl/test/ClassPoolWithRepositoryTestCase.java
   projects/aop/trunk/aspects/src/test/java/org/jboss/test/proxyfactory/AbstractProxyTestDelegate.java
   projects/aop/trunk/aspects/src/test/java/org/jboss/test/proxyfactory/support/TestManagedConnectionFactory.java
   projects/aop/trunk/aspects/src/test/java/org/jboss/test/proxyfactory/test/HollowWithMixinTestCase.java
   projects/aop/trunk/aspects/src/test/java/org/jboss/test/proxyfactory/test/SimpleTestCase.java
Log:
Tidy up and get rid of warnings

Modified: projects/aop/trunk/aop/src/main/java/org/jboss/aop/classpool/AOPClassPool.java
===================================================================
--- projects/aop/trunk/aop/src/main/java/org/jboss/aop/classpool/AOPClassPool.java	2009-02-18 11:15:04 UTC (rev 84373)
+++ projects/aop/trunk/aop/src/main/java/org/jboss/aop/classpool/AOPClassPool.java	2009-02-18 11:18:18 UTC (rev 84374)
@@ -21,11 +21,9 @@
   */
 package org.jboss.aop.classpool;
 
-import java.security.ProtectionDomain;
 import java.util.Map;
 import java.util.concurrent.ConcurrentHashMap;
 
-import javassist.CannotCompileException;
 import javassist.ClassPool;
 import javassist.CtClass;
 import javassist.NotFoundException;

Modified: projects/aop/trunk/aop/src/main/java/org/jboss/aop/reflectprototype/JavassistAopTypeInfoFactoryImpl.java
===================================================================
--- projects/aop/trunk/aop/src/main/java/org/jboss/aop/reflectprototype/JavassistAopTypeInfoFactoryImpl.java	2009-02-18 11:15:04 UTC (rev 84373)
+++ projects/aop/trunk/aop/src/main/java/org/jboss/aop/reflectprototype/JavassistAopTypeInfoFactoryImpl.java	2009-02-18 11:18:18 UTC (rev 84374)
@@ -134,7 +134,6 @@
       throw new RuntimeException("This method is void!!!");
    }
    
-   @SuppressWarnings("unchecked")
    protected Object instantiate(CtClass ctClass)
    {
       try

Modified: projects/aop/trunk/aop/src/test/java/org/jboss/test/aop/standalone/PackageTestCase.java
===================================================================
--- projects/aop/trunk/aop/src/test/java/org/jboss/test/aop/standalone/PackageTestCase.java	2009-02-18 11:15:04 UTC (rev 84373)
+++ projects/aop/trunk/aop/src/test/java/org/jboss/test/aop/standalone/PackageTestCase.java	2009-02-18 11:18:18 UTC (rev 84374)
@@ -21,14 +21,10 @@
   */
 package org.jboss.test.aop.standalone;
 
-import java.lang.ref.WeakReference;
-import java.util.Map;
-
 import junit.framework.Test;
 import junit.framework.TestSuite;
 
 import org.jboss.aop.AspectManager;
-import org.jboss.aop.Domain;
 import org.jboss.test.aop.AOPTestWithSetup;
 
 /**

Modified: projects/aop/trunk/asintegration-core/src/test/java/org/jboss/test/aop/classpool/test/SimpleDelegatingClassPoolTestCase.java
===================================================================
--- projects/aop/trunk/asintegration-core/src/test/java/org/jboss/test/aop/classpool/test/SimpleDelegatingClassPoolTestCase.java	2009-02-18 11:15:04 UTC (rev 84373)
+++ projects/aop/trunk/asintegration-core/src/test/java/org/jboss/test/aop/classpool/test/SimpleDelegatingClassPoolTestCase.java	2009-02-18 11:18:18 UTC (rev 84374)
@@ -122,7 +122,7 @@
    
    private void accessCanLoadCtArray(ClassPool poolA, ClassPool poolB) throws Exception
    {
-      CtClass x = poolA.get(String.class.getName() + "[][]");
+      poolA.get(String.class.getName() + "[][]");
       CtClass a = poolA.get(CLASS_A);
       CtClass aArray = poolA.get(CLASS_A + "[]");
       CtClass bArray = poolA.get(CLASS_B + "[][]");

Modified: projects/aop/trunk/asintegration-jmx/src/main/java/org/jboss/aop/classpool/ucl/JBossUclDelegatingClassPoolFactory.java
===================================================================
--- projects/aop/trunk/asintegration-jmx/src/main/java/org/jboss/aop/classpool/ucl/JBossUclDelegatingClassPoolFactory.java	2009-02-18 11:15:04 UTC (rev 84373)
+++ projects/aop/trunk/asintegration-jmx/src/main/java/org/jboss/aop/classpool/ucl/JBossUclDelegatingClassPoolFactory.java	2009-02-18 11:18:18 UTC (rev 84374)
@@ -30,7 +30,6 @@
 import javassist.scopedpool.ScopedClassPoolFactory;
 import javassist.scopedpool.ScopedClassPoolRepository;
 
-import org.jboss.aop.classpool.AbstractJBossClassPoolFactory;
 import org.jboss.aop.classpool.AbstractJBossDelegatingClassPoolFactory;
 import org.jboss.aop.classpool.BaseClassPoolDomain;
 import org.jboss.aop.classpool.ClassPoolDomain;

Modified: projects/aop/trunk/asintegration-jmx/src/test/java/org/jboss/test/aop/classpool/ucl/test/UclDelegatingClassPoolTestCase.java
===================================================================
--- projects/aop/trunk/asintegration-jmx/src/test/java/org/jboss/test/aop/classpool/ucl/test/UclDelegatingClassPoolTestCase.java	2009-02-18 11:15:04 UTC (rev 84373)
+++ projects/aop/trunk/asintegration-jmx/src/test/java/org/jboss/test/aop/classpool/ucl/test/UclDelegatingClassPoolTestCase.java	2009-02-18 11:18:18 UTC (rev 84374)
@@ -534,7 +534,7 @@
          assertSame(childClazz, childClazzFromChild);
          
 
-         Class<?> clazzA = child.getClassLoader().loadClass(CLASS_A);
+         child.getClassLoader().loadClass(CLASS_A);
       }
       finally
       {

Modified: projects/aop/trunk/asintegration-mc/src/main/java/org/jboss/aop/asintegration/jboss5/JBoss5Integration.java
===================================================================
--- projects/aop/trunk/asintegration-mc/src/main/java/org/jboss/aop/asintegration/jboss5/JBoss5Integration.java	2009-02-18 11:15:04 UTC (rev 84373)
+++ projects/aop/trunk/asintegration-mc/src/main/java/org/jboss/aop/asintegration/jboss5/JBoss5Integration.java	2009-02-18 11:18:18 UTC (rev 84374)
@@ -34,7 +34,6 @@
 import javassist.scopedpool.ScopedClassPoolRepository;
 
 import org.jboss.aop.AspectManager;
-import org.jboss.aop.classpool.AOPClassLoaderScopingPolicy;
 import org.jboss.aop.asintegration.JBossIntegration;
 import org.jboss.classloader.spi.ClassLoaderSystem;
 import org.jboss.classloading.spi.RealClassLoader;

Modified: projects/aop/trunk/asintegration-mc/src/main/java/org/jboss/aop/classpool/jbosscl/JBossClClassPoolDomain.java
===================================================================
--- projects/aop/trunk/asintegration-mc/src/main/java/org/jboss/aop/classpool/jbosscl/JBossClClassPoolDomain.java	2009-02-18 11:15:04 UTC (rev 84373)
+++ projects/aop/trunk/asintegration-mc/src/main/java/org/jboss/aop/classpool/jbosscl/JBossClClassPoolDomain.java	2009-02-18 11:18:18 UTC (rev 84374)
@@ -32,7 +32,6 @@
 
 import javassist.ClassPool;
 import javassist.CtClass;
-import javassist.CtPrimitiveType;
 import javassist.NotFoundException;
 
 import org.jboss.aop.AspectManager;

Modified: projects/aop/trunk/asintegration-mc/src/main/java/org/jboss/aop/classpool/jbosscl/JBossClDelegatingClassPoolFactory.java
===================================================================
--- projects/aop/trunk/asintegration-mc/src/main/java/org/jboss/aop/classpool/jbosscl/JBossClDelegatingClassPoolFactory.java	2009-02-18 11:15:04 UTC (rev 84373)
+++ projects/aop/trunk/asintegration-mc/src/main/java/org/jboss/aop/classpool/jbosscl/JBossClDelegatingClassPoolFactory.java	2009-02-18 11:18:18 UTC (rev 84374)
@@ -28,7 +28,6 @@
 
 import org.jboss.aop.AspectManager;
 import org.jboss.aop.asintegration.jboss5.DomainRegistry;
-import org.jboss.aop.classpool.AbstractJBossClassPoolFactory;
 import org.jboss.aop.classpool.AbstractJBossDelegatingClassPoolFactory;
 import org.jboss.aop.classpool.ClassPoolDomain;
 import org.jboss.aop.classpool.ClassPoolDomainRegistry;

Modified: projects/aop/trunk/asintegration-mc/src/test/java/org/jboss/test/aop/classpool/jbosscl/test/ClassPoolWithRepositoryTestCase.java
===================================================================
--- projects/aop/trunk/asintegration-mc/src/test/java/org/jboss/test/aop/classpool/jbosscl/test/ClassPoolWithRepositoryTestCase.java	2009-02-18 11:15:04 UTC (rev 84373)
+++ projects/aop/trunk/asintegration-mc/src/test/java/org/jboss/test/aop/classpool/jbosscl/test/ClassPoolWithRepositoryTestCase.java	2009-02-18 11:18:18 UTC (rev 84374)
@@ -733,7 +733,7 @@
    
    private void accessCanLoadCtArray(ClassPool poolA, ClassPool poolB) throws Exception
    {
-      CtClass x = poolA.get(String.class.getName() + "[][]");
+      poolA.get(String.class.getName() + "[][]");
       CtClass a = poolA.get(CLASS_A);
       CtClass aArray = poolA.get(CLASS_A + "[]");
       CtClass bArray = poolA.get(CLASS_B + "[][]");

Modified: projects/aop/trunk/aspects/src/test/java/org/jboss/test/proxyfactory/AbstractProxyTestDelegate.java
===================================================================
--- projects/aop/trunk/aspects/src/test/java/org/jboss/test/proxyfactory/AbstractProxyTestDelegate.java	2009-02-18 11:15:04 UTC (rev 84373)
+++ projects/aop/trunk/aspects/src/test/java/org/jboss/test/proxyfactory/AbstractProxyTestDelegate.java	2009-02-18 11:18:18 UTC (rev 84374)
@@ -40,7 +40,7 @@
     * @param clazz the test class
     * @throws Exception for any error
     */
-   public AbstractProxyTestDelegate(Class clazz) throws Exception
+   public AbstractProxyTestDelegate(Class<?> clazz) throws Exception
    {
       super(clazz);
    }

Modified: projects/aop/trunk/aspects/src/test/java/org/jboss/test/proxyfactory/support/TestManagedConnectionFactory.java
===================================================================
--- projects/aop/trunk/aspects/src/test/java/org/jboss/test/proxyfactory/support/TestManagedConnectionFactory.java	2009-02-18 11:15:04 UTC (rev 84373)
+++ projects/aop/trunk/aspects/src/test/java/org/jboss/test/proxyfactory/support/TestManagedConnectionFactory.java	2009-02-18 11:18:18 UTC (rev 84374)
@@ -62,7 +62,7 @@
       throw new org.jboss.util.NotImplementedException("getLogWriter");
    }
 
-   public ManagedConnection matchManagedConnections(Set connectionSet, Subject subject, ConnectionRequestInfo cxRequestInfo) throws ResourceException
+   public ManagedConnection matchManagedConnections(@SuppressWarnings("unchecked")Set connectionSet, Subject subject, ConnectionRequestInfo cxRequestInfo) throws ResourceException
    {
       throw new org.jboss.util.NotImplementedException("matchManagedConnections");
    }

Modified: projects/aop/trunk/aspects/src/test/java/org/jboss/test/proxyfactory/test/HollowWithMixinTestCase.java
===================================================================
--- projects/aop/trunk/aspects/src/test/java/org/jboss/test/proxyfactory/test/HollowWithMixinTestCase.java	2009-02-18 11:15:04 UTC (rev 84373)
+++ projects/aop/trunk/aspects/src/test/java/org/jboss/test/proxyfactory/test/HollowWithMixinTestCase.java	2009-02-18 11:18:18 UTC (rev 84374)
@@ -21,8 +21,6 @@
 */
 package org.jboss.test.proxyfactory.test;
 
-import java.lang.reflect.Method;
-
 import junit.framework.Test;
 
 import org.jboss.aop.proxy.container.AOPProxyFactoryMixin;

Modified: projects/aop/trunk/aspects/src/test/java/org/jboss/test/proxyfactory/test/SimpleTestCase.java
===================================================================
--- projects/aop/trunk/aspects/src/test/java/org/jboss/test/proxyfactory/test/SimpleTestCase.java	2009-02-18 11:15:04 UTC (rev 84373)
+++ projects/aop/trunk/aspects/src/test/java/org/jboss/test/proxyfactory/test/SimpleTestCase.java	2009-02-18 11:18:18 UTC (rev 84374)
@@ -80,7 +80,7 @@
    
    public void testStressDynamicProxy() throws Exception
    {
-      Class[] interfaces = new Class[] { Simple.class };
+      Class<?>[] interfaces = new Class[] { Simple.class };
       ClassLoader cl = Simple.class.getClassLoader();
       for (int i = 0; i < iterations; ++i)
       {




More information about the jboss-cvs-commits mailing list