[jboss-cvs] JBossAS SVN: r85914 - in projects/jboss-reflect/trunk/src/main/java/org/jboss: beans/info/spi/helpers and 7 other directories.

jboss-cvs-commits at lists.jboss.org jboss-cvs-commits at lists.jboss.org
Mon Mar 16 07:39:48 EDT 2009


Author: stalep
Date: 2009-03-16 07:39:48 -0400 (Mon, 16 Mar 2009)
New Revision: 85914

Modified:
   projects/jboss-reflect/trunk/src/main/java/org/jboss/beans/info/plugins/AbstractBeanInfo.java
   projects/jboss-reflect/trunk/src/main/java/org/jboss/beans/info/plugins/AbstractPropertyInfo.java
   projects/jboss-reflect/trunk/src/main/java/org/jboss/beans/info/plugins/AllBeanInfo.java
   projects/jboss-reflect/trunk/src/main/java/org/jboss/beans/info/plugins/FieldBeanInfo.java
   projects/jboss-reflect/trunk/src/main/java/org/jboss/beans/info/plugins/FieldPropertyInfo.java
   projects/jboss-reflect/trunk/src/main/java/org/jboss/beans/info/plugins/GetterAndFieldPropertyInfo.java
   projects/jboss-reflect/trunk/src/main/java/org/jboss/beans/info/plugins/NestedPropertyInfo.java
   projects/jboss-reflect/trunk/src/main/java/org/jboss/beans/info/plugins/SetterAndFieldPropertyInfo.java
   projects/jboss-reflect/trunk/src/main/java/org/jboss/beans/info/spi/helpers/UnmodifiableBeanInfo.java
   projects/jboss-reflect/trunk/src/main/java/org/jboss/config/plugins/BasicConfiguration.java
   projects/jboss-reflect/trunk/src/main/java/org/jboss/config/plugins/property/PropertyConfiguration.java
   projects/jboss-reflect/trunk/src/main/java/org/jboss/config/spi/ConfigurationPermission.java
   projects/jboss-reflect/trunk/src/main/java/org/jboss/joinpoint/plugins/BasicConstructorJoinPoint.java
   projects/jboss-reflect/trunk/src/main/java/org/jboss/joinpoint/plugins/BasicJoinpointFactory.java
   projects/jboss-reflect/trunk/src/main/java/org/jboss/joinpoint/plugins/BasicTargettedJoinPoint.java
   projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/AnnotationAttributeImpl.java
   projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/AnnotationInfoImpl.java
   projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/AnnotationValueImpl.java
   projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/ArrayInfoImpl.java
   projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/ArrayValueImpl.java
   projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/ClassInfoImpl.java
   projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/ClassValueImpl.java
   projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/ConstructorInfoImpl.java
   projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/EnumInfoImpl.java
   projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/EnumValueImpl.java
   projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/FieldInfoImpl.java
   projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/InterfaceInfoImpl.java
   projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/MethodInfoImpl.java
   projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/PackageInfoImpl.java
   projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/ParameterInfoImpl.java
   projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/StringValueImpl.java
   projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/ValueConvertor.java
   projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/introspection/IntrospectionTypeInfoFactoryImpl.java
   projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/introspection/ParameterizedArrayInfo.java
   projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/introspection/ParameterizedClassInfo.java
   projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/introspection/ReflectClassInfoImpl.java
   projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/introspection/ReflectConstructorInfoImpl.java
   projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/introspection/ReflectFieldInfoImpl.java
   projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/introspection/ReflectMethodInfoImpl.java
   projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/javassist/JavassistMethodInfo.java
   projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/javassist/JavassistTypeInfo.java
   projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/javassist/JavassistTypeInfoFactory.java
   projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/javassist/JavassistTypeInfoFactoryImpl.java
Log:
[JBREFLECT-49]
lot of code cleanup. removed warnings.
changed JavassistTypeInfoFactory to implement MutableTypeInfoFactory.

Modified: projects/jboss-reflect/trunk/src/main/java/org/jboss/beans/info/plugins/AbstractBeanInfo.java
===================================================================
--- projects/jboss-reflect/trunk/src/main/java/org/jboss/beans/info/plugins/AbstractBeanInfo.java	2009-03-16 11:39:19 UTC (rev 85913)
+++ projects/jboss-reflect/trunk/src/main/java/org/jboss/beans/info/plugins/AbstractBeanInfo.java	2009-03-16 11:39:48 UTC (rev 85914)
@@ -133,18 +133,18 @@
    /**
     * Get a property
     *
-    * @param name the property name
+    * @param propertyName the property name
     * @return the property
     * @throws IllegalArgumentException if there is no such property
     */
-   public PropertyInfo getProperty(String name)
+   public PropertyInfo getProperty(String propertyName)
    {
-      if (name == null)
+      if (propertyName == null)
          throw new IllegalArgumentException("Null name");
 
-      PropertyInfo property = findPropertyInfo(name);
+      PropertyInfo property = findPropertyInfo(propertyName);
       if (property == null)
-         throw new IllegalArgumentException("No such property " + name + " for bean " + getName() + " available " + propertiesByName.keySet());
+         throw new IllegalArgumentException("No such property " + propertyName + " for bean " + getName() + " available " + propertiesByName.keySet());
       return property;
    }
 
@@ -154,9 +154,9 @@
     * @param name the property name
     * @return the property or null if no such property
     */
-   protected PropertyInfo findPropertyInfo(String name)
+   protected PropertyInfo findPropertyInfo(String propertyName)
    {
-      return propertiesByName.get(name);
+      return propertiesByName.get(propertyName);
    }
 
    /**
@@ -279,9 +279,9 @@
       return newInstance(typeInfosToStrings(paramTypes), params);
    }
 
-   public Object getProperty(Object bean, String name) throws Throwable
+   public Object getProperty(Object bean, String beanName) throws Throwable
    {
-      return BeanInfoUtil.get(this, bean, name);
+      return BeanInfoUtil.get(this, bean, beanName);
    }
 
    public void setProperty(Object bean, String name, Object value) throws Throwable
@@ -289,48 +289,49 @@
       BeanInfoUtil.set(this, bean, name, value);
    }
 
-   public Object invoke(Object bean, String name) throws Throwable
+   public Object invoke(Object bean, String beanName) throws Throwable
    {
-      return invoke(bean, name, (String[]) null, null);
+      return invoke(bean, beanName, (String[]) null, null);
    }
 
-   public Object invoke(Object bean, String name, String[] paramTypes, Object[] params) throws Throwable
+   public Object invoke(Object bean, String beanName, String[] paramTypes, Object[] params) throws Throwable
    {
-      MethodJoinpoint joinpoint = Config.getMethodJoinpoint(bean, getJoinpointFactory(), name, paramTypes, params);
+      MethodJoinpoint joinpoint = Config.getMethodJoinpoint(bean, getJoinpointFactory(), beanName, paramTypes, params);
       return joinpoint.dispatch();
    }
 
-   public Object invoke(Object bean, String name, Class<?>[] paramTypes, Object[] params) throws Throwable
+   public Object invoke(Object bean, String beanName, Class<?>[] paramTypes, Object[] params) throws Throwable
    {
-      return invoke(bean, name, classesToStrings(paramTypes), params);
+      return invoke(bean, beanName, classesToStrings(paramTypes), params);
    }
 
-   public Object invoke(Object bean, String name, TypeInfo[] paramTypes, Object[] params) throws Throwable
+   public Object invoke(Object bean, String beanName, TypeInfo[] paramTypes, Object[] params) throws Throwable
    {
-      return invoke(bean, name, typeInfosToStrings(paramTypes), params);
+      return invoke(bean, beanName, typeInfosToStrings(paramTypes), params);
    }
 
-   public Object invokeStatic(String name) throws Throwable
+   public Object invokeStatic(String beanName) throws Throwable
    {
-      return invokeStatic(name, (String[]) null, null);
+      return invokeStatic(beanName, (String[]) null, null);
    }
 
-   public Object invokeStatic(String name, String[] paramTypes, Object[] params) throws Throwable
+   public Object invokeStatic(String beanName, String[] paramTypes, Object[] params) throws Throwable
    {
-      MethodJoinpoint joinpoint = Config.getStaticMethodJoinpoint(getJoinpointFactory(), name, paramTypes, params);
+      MethodJoinpoint joinpoint = Config.getStaticMethodJoinpoint(getJoinpointFactory(), beanName, paramTypes, params);
       return joinpoint.dispatch();
    }
 
-   public Object invokeStatic(String name, Class<?>[] paramTypes, Object[] params) throws Throwable
+   public Object invokeStatic(String beanName, Class<?>[] paramTypes, Object[] params) throws Throwable
    {
-      return invokeStatic(name, classesToStrings(paramTypes), params);
+      return invokeStatic(beanName, classesToStrings(paramTypes), params);
    }
 
-   public Object invokeStatic(String name, TypeInfo[] paramTypes, Object[] params) throws Throwable
+   public Object invokeStatic(String beanName, TypeInfo[] paramTypes, Object[] params) throws Throwable
    {
-      return invokeStatic(name, typeInfosToStrings(paramTypes), params);
+      return invokeStatic(beanName, typeInfosToStrings(paramTypes), params);
    }
 
+   @Override
    public boolean equals(Object object)
    {
       if (object == null || object instanceof AbstractBeanInfo == false)
@@ -352,6 +353,7 @@
       return true;
    }
    
+   @Override
    public void toString(JBossStringBuilder buffer)
    {
       buffer.append("name=").append(name);
@@ -367,11 +369,13 @@
       list(buffer, events);
    }
    
+   @Override
    public void toShortString(JBossStringBuilder buffer)
    {
       buffer.append(name);
    }
    
+   @Override
    public int getHashCode()
    {
       return name.hashCode();

Modified: projects/jboss-reflect/trunk/src/main/java/org/jboss/beans/info/plugins/AbstractPropertyInfo.java
===================================================================
--- projects/jboss-reflect/trunk/src/main/java/org/jboss/beans/info/plugins/AbstractPropertyInfo.java	2009-03-16 11:39:19 UTC (rev 85913)
+++ projects/jboss-reflect/trunk/src/main/java/org/jboss/beans/info/plugins/AbstractPropertyInfo.java	2009-03-16 11:39:48 UTC (rev 85914)
@@ -103,15 +103,15 @@
    /**
     * Initialize fields.
     *
-    * @param name the name
-    * @param upperName the upper name
-    * @param type the type
+    * @param fieldName the name
+    * @param fieldUpperName the upper name
+    * @param fieldType the type
     */
-   protected void init(String name, String upperName, TypeInfo type)
+   protected void init(String fieldName, String fieldUpperName, TypeInfo fieldType)
    {
-      this.name = name;
-      this.upperName = upperName;
-      this.type = type;
+      this.name = fieldName;
+      this.upperName = fieldUpperName;
+      this.type = fieldType;
    }
 
    public BeanInfo getBeanInfo()

Modified: projects/jboss-reflect/trunk/src/main/java/org/jboss/beans/info/plugins/AllBeanInfo.java
===================================================================
--- projects/jboss-reflect/trunk/src/main/java/org/jboss/beans/info/plugins/AllBeanInfo.java	2009-03-16 11:39:19 UTC (rev 85913)
+++ projects/jboss-reflect/trunk/src/main/java/org/jboss/beans/info/plugins/AllBeanInfo.java	2009-03-16 11:39:48 UTC (rev 85914)
@@ -49,6 +49,7 @@
       super(beanInfoFactory, classAdapter, properties, constructors, methods, events);
    }
 
+   @Override
    protected FieldFilter getFieldFilter()
    {
       return FieldFilter.ALL;

Modified: projects/jboss-reflect/trunk/src/main/java/org/jboss/beans/info/plugins/FieldBeanInfo.java
===================================================================
--- projects/jboss-reflect/trunk/src/main/java/org/jboss/beans/info/plugins/FieldBeanInfo.java	2009-03-16 11:39:19 UTC (rev 85913)
+++ projects/jboss-reflect/trunk/src/main/java/org/jboss/beans/info/plugins/FieldBeanInfo.java	2009-03-16 11:39:48 UTC (rev 85914)
@@ -58,6 +58,7 @@
       super(beanInfoFactory, classAdapter, properties, constructors, methods, events);
    }
 
+   @Override
    public void setProperties(Set<PropertyInfo> properties)
    {
       setFields(getFields(classAdapter.getClassInfo(), getFieldFilter()));
@@ -71,6 +72,7 @@
       }
    }
 
+   @Override
    protected PropertyInfo replaceProperty(PropertyInfo original)
    {
       String name = original.getName();

Modified: projects/jboss-reflect/trunk/src/main/java/org/jboss/beans/info/plugins/FieldPropertyInfo.java
===================================================================
--- projects/jboss-reflect/trunk/src/main/java/org/jboss/beans/info/plugins/FieldPropertyInfo.java	2009-03-16 11:39:19 UTC (rev 85913)
+++ projects/jboss-reflect/trunk/src/main/java/org/jboss/beans/info/plugins/FieldPropertyInfo.java	2009-03-16 11:39:48 UTC (rev 85914)
@@ -99,6 +99,7 @@
       field.set(bean, value);
    }
 
+   @Override
    public FieldInfo getFieldInfo()
    {
       return field;

Modified: projects/jboss-reflect/trunk/src/main/java/org/jboss/beans/info/plugins/GetterAndFieldPropertyInfo.java
===================================================================
--- projects/jboss-reflect/trunk/src/main/java/org/jboss/beans/info/plugins/GetterAndFieldPropertyInfo.java	2009-03-16 11:39:19 UTC (rev 85913)
+++ projects/jboss-reflect/trunk/src/main/java/org/jboss/beans/info/plugins/GetterAndFieldPropertyInfo.java	2009-03-16 11:39:48 UTC (rev 85914)
@@ -48,16 +48,19 @@
       setupAnnotations(mergeAnnotations(field.getAnnotations(), previous.getAnnotations()));
    }
 
+   @Override
    public Object get(Object bean) throws Throwable
    {
       return previous.get(bean);
    }
 
+   @Override
    public MethodInfo getGetter()
    {
       return previous.getGetter();
    }
 
+   @Override
    public void setGetter(MethodInfo getter)
    {
       previous.setGetter(getter);

Modified: projects/jboss-reflect/trunk/src/main/java/org/jboss/beans/info/plugins/NestedPropertyInfo.java
===================================================================
--- projects/jboss-reflect/trunk/src/main/java/org/jboss/beans/info/plugins/NestedPropertyInfo.java	2009-03-16 11:39:19 UTC (rev 85913)
+++ projects/jboss-reflect/trunk/src/main/java/org/jboss/beans/info/plugins/NestedPropertyInfo.java	2009-03-16 11:39:48 UTC (rev 85914)
@@ -201,12 +201,12 @@
       throw new IllegalArgumentException("Unable to determine right PropertyInfo on " + beanInfo + " by name: " + name);
    }
 
-   public AnnotationValue getAnnotation(String name)
+   public AnnotationValue getAnnotation(String annotationName)
    {
       throw new IllegalArgumentException("Unable to determine right PropertyInfo on " + beanInfo + " by name: " + this.name);
    }
 
-   public boolean isAnnotationPresent(String name)
+   public boolean isAnnotationPresent(String annotationName)
    {
       throw new IllegalArgumentException("Unable to determine right PropertyInfo on " + beanInfo + " by name: " + this.name);
    }

Modified: projects/jboss-reflect/trunk/src/main/java/org/jboss/beans/info/plugins/SetterAndFieldPropertyInfo.java
===================================================================
--- projects/jboss-reflect/trunk/src/main/java/org/jboss/beans/info/plugins/SetterAndFieldPropertyInfo.java	2009-03-16 11:39:19 UTC (rev 85913)
+++ projects/jboss-reflect/trunk/src/main/java/org/jboss/beans/info/plugins/SetterAndFieldPropertyInfo.java	2009-03-16 11:39:48 UTC (rev 85914)
@@ -48,16 +48,19 @@
       setupAnnotations(mergeAnnotations(field.getAnnotations(), previous.getAnnotations()));
    }
 
+   @Override
    public void set(Object bean, Object value) throws Throwable
    {
       previous.set(bean, value);
    }
 
+   @Override
    public MethodInfo getSetter()
    {
       return previous.getSetter();
    }
 
+   @Override
    public void setSetter(MethodInfo setter)
    {
       previous.setSetter(setter);

Modified: projects/jboss-reflect/trunk/src/main/java/org/jboss/beans/info/spi/helpers/UnmodifiableBeanInfo.java
===================================================================
--- projects/jboss-reflect/trunk/src/main/java/org/jboss/beans/info/spi/helpers/UnmodifiableBeanInfo.java	2009-03-16 11:39:19 UTC (rev 85913)
+++ projects/jboss-reflect/trunk/src/main/java/org/jboss/beans/info/spi/helpers/UnmodifiableBeanInfo.java	2009-03-16 11:39:48 UTC (rev 85914)
@@ -186,11 +186,13 @@
       return delegate.invokeStatic(name, paramTypes, params);
    }
 
+   @Override
    public String toShortString()
    {
       return delegate.toShortString();
    }
 
+   @Override
    public void toShortString(JBossStringBuilder buffer)
    {
       delegate.toShortString(buffer);

Modified: projects/jboss-reflect/trunk/src/main/java/org/jboss/config/plugins/BasicConfiguration.java
===================================================================
--- projects/jboss-reflect/trunk/src/main/java/org/jboss/config/plugins/BasicConfiguration.java	2009-03-16 11:39:19 UTC (rev 85913)
+++ projects/jboss-reflect/trunk/src/main/java/org/jboss/config/plugins/BasicConfiguration.java	2009-03-16 11:39:48 UTC (rev 85914)
@@ -45,11 +45,13 @@
    {
    }
    
+   @Override
    protected BeanInfoFactory createDefaultBeanInfoFactory() throws Throwable
    {
       return new AbstractBeanInfoFactory();
    }
 
+   @Override
    protected ClassAdapterFactory createDefaultClassAdapterFactory() throws Throwable
    {
       BasicClassAdapterFactory result = new BasicClassAdapterFactory();
@@ -57,11 +59,13 @@
       return result;
    }
 
+   @Override
    protected TypeInfoFactory createDefaultTypeInfoFactory() throws Throwable
    {
       return new IntrospectionTypeInfoFactory();
    }
 
+   @Override
    protected JoinpointFactoryBuilder createDefaultJoinpointFactoryBuilder() throws Throwable
    {
       return new BasicJoinpointFactoryBuilder();

Modified: projects/jboss-reflect/trunk/src/main/java/org/jboss/config/plugins/property/PropertyConfiguration.java
===================================================================
--- projects/jboss-reflect/trunk/src/main/java/org/jboss/config/plugins/property/PropertyConfiguration.java	2009-03-16 11:39:19 UTC (rev 85913)
+++ projects/jboss-reflect/trunk/src/main/java/org/jboss/config/plugins/property/PropertyConfiguration.java	2009-03-16 11:39:48 UTC (rev 85914)
@@ -75,11 +75,13 @@
       return properties;
    }
    
+   @Override
    protected BeanInfoFactory createDefaultBeanInfoFactory() throws Throwable
    {
       return (BeanInfoFactory) loadFromProperties(PropertyConfigurationConstants.BEAN_INFO_FACTORY_NAME, PropertyConfigurationConstants.BEAN_INFO_FACTORY_DEFAULT, BeanInfoFactory.class);
    }
    
+   @Override
    protected ClassAdapterFactory createDefaultClassAdapterFactory() throws Throwable
    {
       ClassAdapterFactory result = (ClassAdapterFactory) loadFromProperties(PropertyConfigurationConstants.CLASS_ADAPTER_FACTORY_NAME, PropertyConfigurationConstants.CLASS_ADAPTER_FACTORY_DEFAULT, ClassAdapterFactory.class);
@@ -87,11 +89,13 @@
       return result;
    }
 
+   @Override
    protected TypeInfoFactory createDefaultTypeInfoFactory() throws Throwable
    {
       return (TypeInfoFactory) loadFromProperties(PropertyConfigurationConstants.TYPE_INFO_FACTORY_NAME, PropertyConfigurationConstants.TYPE_INFO_FACTORY_DEFAULT, TypeInfoFactory.class);
    }
 
+   @Override
    protected JoinpointFactoryBuilder createDefaultJoinpointFactoryBuilder() throws Throwable
    {
       return (JoinpointFactoryBuilder) loadFromProperties(PropertyConfigurationConstants.JOIN_POINT_FACTORY_BUILDER_NAME, PropertyConfigurationConstants.JOIN_POINT_FACTORY_BUILDER_DEFAULT, JoinpointFactoryBuilder.class);

Modified: projects/jboss-reflect/trunk/src/main/java/org/jboss/config/spi/ConfigurationPermission.java
===================================================================
--- projects/jboss-reflect/trunk/src/main/java/org/jboss/config/spi/ConfigurationPermission.java	2009-03-16 11:39:19 UTC (rev 85913)
+++ projects/jboss-reflect/trunk/src/main/java/org/jboss/config/spi/ConfigurationPermission.java	2009-03-16 11:39:48 UTC (rev 85914)
@@ -76,6 +76,7 @@
    /**
     * @return human readable string.
     */
+   @Override
    public String toString()
    {
       StringBuffer buffer = new StringBuffer(100);
@@ -94,6 +95,7 @@
     * @param p the permission
     * @return true when the permission is implied
     */ 
+   @Override
    public boolean implies(Permission p)
    {
       if( (p instanceof ConfigurationPermission) == false)
@@ -107,6 +109,7 @@
     * 
     * @return the permission collection
     */ 
+   @Override
    public PermissionCollection newPermissionCollection()
    {
       return new ConfigurationPermissionCollection();
@@ -153,6 +156,7 @@
       /** Whether we have all permissions */
       private boolean hasAll;
 
+      @Override
       public void add(Permission p)
       {
          if (isReadOnly())
@@ -165,6 +169,7 @@
             hasAll = true;
       }
 
+      @Override
       public boolean implies(Permission p)
       {
          boolean implies = false;
@@ -177,6 +182,7 @@
          return implies;
       }
 
+      @Override
       public Enumeration<Permission> elements()
       {
          final Iterator<Permission> iter = permissions.iterator();

Modified: projects/jboss-reflect/trunk/src/main/java/org/jboss/joinpoint/plugins/BasicConstructorJoinPoint.java
===================================================================
--- projects/jboss-reflect/trunk/src/main/java/org/jboss/joinpoint/plugins/BasicConstructorJoinPoint.java	2009-03-16 11:39:19 UTC (rev 85913)
+++ projects/jboss-reflect/trunk/src/main/java/org/jboss/joinpoint/plugins/BasicConstructorJoinPoint.java	2009-03-16 11:39:48 UTC (rev 85914)
@@ -63,6 +63,7 @@
       this.arguments = args;
    }
 
+   @Override
    public Object clone()
    {
       try

Modified: projects/jboss-reflect/trunk/src/main/java/org/jboss/joinpoint/plugins/BasicJoinpointFactory.java
===================================================================
--- projects/jboss-reflect/trunk/src/main/java/org/jboss/joinpoint/plugins/BasicJoinpointFactory.java	2009-03-16 11:39:19 UTC (rev 85913)
+++ projects/jboss-reflect/trunk/src/main/java/org/jboss/joinpoint/plugins/BasicJoinpointFactory.java	2009-03-16 11:39:48 UTC (rev 85914)
@@ -79,6 +79,7 @@
       return new BasicMethodJoinPoint(methodInfo);
    }
    
+   @Override
    public void toString(JBossStringBuilder buffer)
    {
       buffer.append("classInfo=").append(classInfo.getName());

Modified: projects/jboss-reflect/trunk/src/main/java/org/jboss/joinpoint/plugins/BasicTargettedJoinPoint.java
===================================================================
--- projects/jboss-reflect/trunk/src/main/java/org/jboss/joinpoint/plugins/BasicTargettedJoinPoint.java	2009-03-16 11:39:19 UTC (rev 85913)
+++ projects/jboss-reflect/trunk/src/main/java/org/jboss/joinpoint/plugins/BasicTargettedJoinPoint.java	2009-03-16 11:39:48 UTC (rev 85914)
@@ -44,6 +44,7 @@
       this.target = target;
    }
 
+   @Override
    public Object clone()
    {
       try

Modified: projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/AnnotationAttributeImpl.java
===================================================================
--- projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/AnnotationAttributeImpl.java	2009-03-16 11:39:19 UTC (rev 85913)
+++ projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/AnnotationAttributeImpl.java	2009-03-16 11:39:48 UTC (rev 85914)
@@ -88,6 +88,7 @@
       return defaultValue;
    }
 
+   @Override
    public boolean equals(Object obj)
    {
       if (this == obj)
@@ -105,6 +106,7 @@
       return true;
    }
 
+   @Override
    public int hashCode()
    {
       return hash;

Modified: projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/AnnotationInfoImpl.java
===================================================================
--- projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/AnnotationInfoImpl.java	2009-03-16 11:39:19 UTC (rev 85913)
+++ projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/AnnotationInfoImpl.java	2009-03-16 11:39:48 UTC (rev 85914)
@@ -81,11 +81,13 @@
       }
    }
 
+   @Override
    public boolean isAnnotation()
    {
       return true;
    }
    
+   @Override
    public ModifierInfo getModifiers()
    {
       return modifiers;
@@ -96,13 +98,14 @@
       return attributes;
    }
 
-   public AnnotationAttribute getAttribute(String name)
+   public AnnotationAttribute getAttribute(String annotationName)
    {
       if (attributeMap == null)
          return null;
-      return attributeMap.get(name);
+      return attributeMap.get(annotationName);
    }
 
+   @Override
    public boolean equals(Object o)
    {
       if (this == o) return true;
@@ -115,6 +118,7 @@
       return true;
    }
 
+   @Override
    public int hashCode()
    {
       return name.hashCode();

Modified: projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/AnnotationValueImpl.java
===================================================================
--- projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/AnnotationValueImpl.java	2009-03-16 11:39:19 UTC (rev 85913)
+++ projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/AnnotationValueImpl.java	2009-03-16 11:39:48 UTC (rev 85914)
@@ -125,11 +125,12 @@
       return underlying;
    }
 
-   public <T extends Annotation> T getUnderlyingAnnotation(Class<T> annotationType)
+   public <T extends Annotation> T getUnderlyingAnnotation(Class<T> type)
    {
-      return annotationType.cast(underlying);
+      return type.cast(underlying);
    }
 
+   @Override
    public boolean equals(Object o)
    {
       if (this == o) return true;
@@ -148,6 +149,7 @@
       return underlying.equals(otherUnderlying);
    }
 
+   @Override
    public int hashCode()
    {
       return hash;
@@ -164,11 +166,13 @@
       hash = result;
    }
 
+   @Override
    public void toShortString(JBossStringBuilder buffer)
    {
       buffer.append(annotationType.getName());
    }
 
+   @Override
    protected void toString(JBossStringBuilder buffer)
    {
       buffer.append("name=").append(annotationType.getName());

Modified: projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/ArrayInfoImpl.java
===================================================================
--- projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/ArrayInfoImpl.java	2009-03-16 11:39:19 UTC (rev 85913)
+++ projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/ArrayInfoImpl.java	2009-03-16 11:39:48 UTC (rev 85914)
@@ -85,8 +85,8 @@
       calculateHash();
    }
 
+   @Override
    @Deprecated
-   @SuppressWarnings("unchecked")
    public Class<? extends Object> getType()
    {
       if (annotatedElement == null)
@@ -104,26 +104,31 @@
       return (Class<? extends Object>)annotatedElement;
    }
 
+   @Override
    public TypeInfo getComponentType()
    {
       return componentType;
    }
    
-   public AnnotationValue getAnnotation(String name)
+   @Override
+   public AnnotationValue getAnnotation(String annotationName)
    {
       return null;
    }
 
+   @Override
    public AnnotationValue[] getAnnotations()
    {
       return UNKNOWN_ANNOTATIONS;
    }
 
-   public boolean isAnnotationPresent(String name)
+   @Override
+   public boolean isAnnotationPresent(String annotationName)
    {
       return false;
    }
 
+   @Override
    public boolean equals(Object o)
    {
       if (this == o) return true;
@@ -137,6 +142,7 @@
       return true;
    }
 
+   @Override
    public int hashCode() { return hash; }
 
    /**

Modified: projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/ArrayValueImpl.java
===================================================================
--- projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/ArrayValueImpl.java	2009-03-16 11:39:19 UTC (rev 85913)
+++ projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/ArrayValueImpl.java	2009-03-16 11:39:48 UTC (rev 85914)
@@ -85,6 +85,7 @@
       return type;
    }
 
+   @Override
    public boolean equals(Object o)
    {
       if (this == o) return true;
@@ -98,6 +99,7 @@
       return true;
    }
 
+   @Override
    public int hashCode()
    {
       return hash;

Modified: projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/ClassInfoImpl.java
===================================================================
--- projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/ClassInfoImpl.java	2009-03-16 11:39:19 UTC (rev 85913)
+++ projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/ClassInfoImpl.java	2009-03-16 11:39:48 UTC (rev 85914)
@@ -22,7 +22,6 @@
 package org.jboss.reflect.plugins;
 
 import java.lang.reflect.Array;
-import java.lang.reflect.Modifier;
 import java.util.Collection;
 import java.util.HashMap;
 import java.util.Map;
@@ -382,11 +381,11 @@
       return genericInterfaces;
    }
    
-   public MethodInfo getDeclaredMethod(String name, TypeInfo[] parameters)
+   public MethodInfo getDeclaredMethod(String methodName, TypeInfo[] parameters)
    {
       if (methods == UNKNOWN_METHODS)
          setDeclaredMethods(classInfoHelper.getMethods(this));
-      return findMethod(methods, name, parameters);
+      return findMethod(methods, methodName, parameters);
    }
 
    public MethodInfo[] getDeclaredMethods()
@@ -396,13 +395,13 @@
       return methods;
    }
 
-   public FieldInfo getDeclaredField(String name)
+   public FieldInfo getDeclaredField(String fieldName)
    {
       if (fields == UNKNOWN_FIELDS)
          setDeclaredFields(classInfoHelper.getFields(this));
       if (fieldMap == null)
          return null;
-      return fieldMap.get(name);
+      return fieldMap.get(fieldName);
    }
 
    public FieldInfo[] getDeclaredFields()
@@ -471,7 +470,6 @@
    }
 
    @Deprecated
-   @SuppressWarnings("unchecked")
    public Class<? extends Object> getType()
    {
       return (Class<? extends Object>) annotatedElement;
@@ -545,7 +543,6 @@
       return getType().isAssignableFrom(info.getType());
    }
 
-   @SuppressWarnings("deprecation")
    public boolean isInstance(Object object)
    {
       return getType().isInstance(object);
@@ -623,16 +620,16 @@
       return expectedType.cast(result);
    }
 
-   public Object getAttachment(String name)
+   public Object getAttachment(String attachName)
    {
-      if (name == null)
+      if (attachName == null)
          throw new IllegalArgumentException("Null name");
       synchronized (this)
       {
          if (attachments == null)
             return null;
       }
-      return attachments.getAttachment(name);
+      return attachments.getAttachment(attachName);
    }
 
    @Override
@@ -657,8 +654,8 @@
 
       final ClassInfo other = (ClassInfo) obj;
 
-      String name = getName();
-      if (name != null ? name.equals(other.getName()) == false : other.getName() != null)
+      String thisName = getName();
+      if (thisName != null ? thisName.equals(other.getName()) == false : other.getName() != null)
          return false;
       return true;
    }
@@ -669,9 +666,9 @@
       return (name != null ? name.hashCode() : 0);
    }
 
-   Object readResolve()
+   protected Object readResolve()
    {
-      TypeInfoFactory typeInfoFactory = SerializationHelper.getTypeInfoFactory();
-      return typeInfoFactory.getTypeInfo(getType());
+      TypeInfoFactory tif = SerializationHelper.getTypeInfoFactory();
+      return tif.getTypeInfo(getType());
    }
 }

Modified: projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/ClassValueImpl.java
===================================================================
--- projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/ClassValueImpl.java	2009-03-16 11:39:19 UTC (rev 85913)
+++ projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/ClassValueImpl.java	2009-03-16 11:39:48 UTC (rev 85914)
@@ -81,6 +81,7 @@
       return type;
    }
 
+   @Override
    public boolean equals(Object o)
    {
       if (this == o) return true;
@@ -94,6 +95,7 @@
       return true;
    }
 
+   @Override
    public int hashCode() { return hash; }
 
    /**

Modified: projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/ConstructorInfoImpl.java
===================================================================
--- projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/ConstructorInfoImpl.java	2009-03-16 11:39:19 UTC (rev 85913)
+++ projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/ConstructorInfoImpl.java	2009-03-16 11:39:48 UTC (rev 85914)
@@ -21,7 +21,6 @@
 */
 package org.jboss.reflect.plugins;
 
-import java.lang.reflect.Modifier;
 import java.util.Arrays;
 
 import org.jboss.reflect.spi.AnnotationValue;
@@ -182,11 +181,13 @@
       throw new NotImplementedException("newInstance");
    }
 
+   @Override
    protected void toString(JBossStringBuilder buffer)
    {
       buffer.append(Arrays.asList(parameterTypes));
    }
 
+   @Override
    public boolean equals(Object obj)
    {
       if (this == obj) 
@@ -201,6 +202,7 @@
       return (Arrays.equals(parameterTypes, other.getParameterTypes()));
    }
 
+   @Override
    public int hashCode()
    {
       return hash;

Modified: projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/EnumInfoImpl.java
===================================================================
--- projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/EnumInfoImpl.java	2009-03-16 11:39:19 UTC (rev 85913)
+++ projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/EnumInfoImpl.java	2009-03-16 11:39:48 UTC (rev 85914)
@@ -79,15 +79,15 @@
       return enumConstants;
    }
 
-   public EnumConstantInfo getEnumConstant(String name)
+   public EnumConstantInfo getEnumConstant(String enumName)
    {
-      return constants.get(name);
+      return constants.get(enumName);
    }
 
    @SuppressWarnings({"unchecked", "deprecation"})
-   public Object getEnumValue(String name)
+   public Object getEnumValue(String enumName)
    {
-      return Enum.valueOf((Class<Enum>) getType(), name);
+      return Enum.valueOf((Class<Enum>) getType(), enumName);
    }
 
    @Override

Modified: projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/EnumValueImpl.java
===================================================================
--- projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/EnumValueImpl.java	2009-03-16 11:39:19 UTC (rev 85913)
+++ projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/EnumValueImpl.java	2009-03-16 11:39:48 UTC (rev 85914)
@@ -81,6 +81,7 @@
       return type;
    }
 
+   @Override
    public boolean equals(Object o)
    {
       if (this == o) return true;
@@ -94,6 +95,7 @@
       return true;
    }
 
+   @Override
    public int hashCode()
    {
       return hash;

Modified: projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/FieldInfoImpl.java
===================================================================
--- projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/FieldInfoImpl.java	2009-03-16 11:39:19 UTC (rev 85913)
+++ projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/FieldInfoImpl.java	2009-03-16 11:39:48 UTC (rev 85914)
@@ -21,8 +21,6 @@
 */
 package org.jboss.reflect.plugins;
 
-import java.lang.reflect.Modifier;
-
 import org.jboss.reflect.spi.AnnotationValue;
 import org.jboss.reflect.spi.ClassInfo;
 import org.jboss.reflect.spi.FieldInfo;
@@ -128,11 +126,13 @@
       throw new NotImplementedException("set");
    }
 
+   @Override
    protected void toString(JBossStringBuilder buffer)
    {
       buffer.append("name=").append(name);
    }
 
+   @Override
    public boolean equals(Object obj)
    {
       if (this == obj) return true;
@@ -146,6 +146,7 @@
       return declaringClass.equals(other.getDeclaringClass());
    }
 
+   @Override
    public int hashCode()
    {
       return hash;

Modified: projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/InterfaceInfoImpl.java
===================================================================
--- projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/InterfaceInfoImpl.java	2009-03-16 11:39:19 UTC (rev 85913)
+++ projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/InterfaceInfoImpl.java	2009-03-16 11:39:48 UTC (rev 85914)
@@ -75,6 +75,7 @@
       super(name, modifiers, interfaces, null);
    }
    
+   @Override
    public boolean isInterface()
    {
       return true;

Modified: projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/MethodInfoImpl.java
===================================================================
--- projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/MethodInfoImpl.java	2009-03-16 11:39:19 UTC (rev 85913)
+++ projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/MethodInfoImpl.java	2009-03-16 11:39:48 UTC (rev 85914)
@@ -21,7 +21,6 @@
 */
 package org.jboss.reflect.plugins;
 
-import java.lang.reflect.Modifier;
 import java.util.Arrays;
 
 import org.jboss.reflect.spi.AnnotationValue;
@@ -205,6 +204,7 @@
       throw new NotImplementedException("invoke");
    }
 
+   @Override
    protected void toString(JBossStringBuilder buffer)
    {
       buffer.append("name=").append(name);
@@ -212,11 +212,13 @@
       buffer.append(" return=").append(returnType);
    }
    
+   @Override
    public void toShortString(JBossStringBuilder buffer)
    {
       buffer.append(name);
    }
 
+   @Override
    public boolean equals(Object obj)
    {
       if (this == obj) return true;
@@ -230,6 +232,7 @@
       return Arrays.equals(parameterTypes, other.getParameterTypes());
    }
 
+   @Override
    public int hashCode()
    {
       return hash;

Modified: projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/PackageInfoImpl.java
===================================================================
--- projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/PackageInfoImpl.java	2009-03-16 11:39:19 UTC (rev 85913)
+++ projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/PackageInfoImpl.java	2009-03-16 11:39:48 UTC (rev 85914)
@@ -90,8 +90,8 @@
 
       final PackageInfo other = (PackageInfo) obj;
 
-      String name = getName();
-      if (name != null ? name.equals(other.getName()) == false : other.getName() != null)
+      String thisName = getName();
+      if (thisName != null ? thisName.equals(other.getName()) == false : other.getName() != null)
          return false;
       return true;
    }

Modified: projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/ParameterInfoImpl.java
===================================================================
--- projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/ParameterInfoImpl.java	2009-03-16 11:39:19 UTC (rev 85913)
+++ projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/ParameterInfoImpl.java	2009-03-16 11:39:48 UTC (rev 85914)
@@ -72,6 +72,7 @@
       return parameterType;
    }
 
+   @Override
    public boolean equals(Object obj)
    {
       if (this == obj)
@@ -83,6 +84,7 @@
       return parameterType.equals(other.getParameterType());
    }
    
+   @Override
    public int hashCode()
    {
       return parameterType.hashCode();

Modified: projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/StringValueImpl.java
===================================================================
--- projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/StringValueImpl.java	2009-03-16 11:39:19 UTC (rev 85913)
+++ projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/StringValueImpl.java	2009-03-16 11:39:48 UTC (rev 85914)
@@ -77,6 +77,7 @@
       return type;
    }
 
+   @Override
    public boolean equals(Object o)
    {
       if (this == o) return true;
@@ -90,6 +91,7 @@
       return true;
    }
 
+   @Override
    public int hashCode()
    {
       int result;

Modified: projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/ValueConvertor.java
===================================================================
--- projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/ValueConvertor.java	2009-03-16 11:39:19 UTC (rev 85913)
+++ projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/ValueConvertor.java	2009-03-16 11:39:48 UTC (rev 85914)
@@ -84,7 +84,6 @@
     * @return the value or null if there is no editor
     * @throws Throwable for any error
     */
-   @SuppressWarnings("unchecked")
    public static Object convertValue(Class<? extends Object> clazz, Object value, boolean replaceProperties) throws Throwable
    {
       return convertValue(clazz, value, replaceProperties, false);

Modified: projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/introspection/IntrospectionTypeInfoFactoryImpl.java
===================================================================
--- projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/introspection/IntrospectionTypeInfoFactoryImpl.java	2009-03-16 11:39:19 UTC (rev 85913)
+++ projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/introspection/IntrospectionTypeInfoFactoryImpl.java	2009-03-16 11:39:48 UTC (rev 85914)
@@ -421,6 +421,7 @@
       return getTypeInfo(clazz);
    }
 
+   @Override
    protected TypeInfo instantiate(Class<?> clazz)
    {
       ClassInfoImpl result;
@@ -465,6 +466,7 @@
       return result;
    }
 
+   @Override
    protected TypeInfo instantiate(ParameterizedType type)
    {
       Class<?> rawType = (Class<?>) type.getRawType();
@@ -474,11 +476,13 @@
       return new ParameterizedClassInfo(this, rawTypeInfo, type);
    }
 
+   @Override
    protected void generate(Class<?> clazz, TypeInfo result)
    {
       generateTypeInfo(clazz, (ClassInfoImpl) result);
    }
 
+   @Override
    protected void generate(ParameterizedType type, TypeInfo result)
    {
       // Everything is lazy

Modified: projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/introspection/ParameterizedArrayInfo.java
===================================================================
--- projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/introspection/ParameterizedArrayInfo.java	2009-03-16 11:39:19 UTC (rev 85913)
+++ projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/introspection/ParameterizedArrayInfo.java	2009-03-16 11:39:48 UTC (rev 85914)
@@ -49,6 +49,7 @@
       super(factory, delegate, parameterizedType);
    }
 
+   @Override
    public TypeInfo getComponentType()
    {
       return delegate.getComponentType();

Modified: projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/introspection/ParameterizedClassInfo.java
===================================================================
--- projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/introspection/ParameterizedClassInfo.java	2009-03-16 11:39:19 UTC (rev 85913)
+++ projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/introspection/ParameterizedClassInfo.java	2009-03-16 11:39:48 UTC (rev 85914)
@@ -77,6 +77,7 @@
       this.parameterizedType = parameterizedType;
    }
 
+   @Override
    public TypeInfoFactory getTypeInfoFactory()
    {
       return factory;

Modified: projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/introspection/ReflectClassInfoImpl.java
===================================================================
--- projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/introspection/ReflectClassInfoImpl.java	2009-03-16 11:39:19 UTC (rev 85913)
+++ projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/introspection/ReflectClassInfoImpl.java	2009-03-16 11:39:48 UTC (rev 85914)
@@ -68,16 +68,18 @@
       super(name, modifiers, interfaces, superclass);
    }
 
+   @Override
    @SuppressWarnings("deprecation")
    public boolean isInterface()
    {
       return getType().isInterface();
    }
 
+   @Override
    @SuppressWarnings("deprecation")
-   Object readResolve()
+   protected Object readResolve()
    {
-      TypeInfoFactory typeInfoFactory = IntrospectionTypeInfoFactory.getDelegate();
-      return typeInfoFactory.getTypeInfo(getType());
+      TypeInfoFactory tif = IntrospectionTypeInfoFactory.getDelegate();
+      return tif.getTypeInfo(getType());
    }
 }

Modified: projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/introspection/ReflectConstructorInfoImpl.java
===================================================================
--- projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/introspection/ReflectConstructorInfoImpl.java	2009-03-16 11:39:19 UTC (rev 85913)
+++ projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/introspection/ReflectConstructorInfoImpl.java	2009-03-16 11:39:48 UTC (rev 85914)
@@ -102,6 +102,7 @@
       return constructor;
    }
 
+   @Override
    public Object newInstance(Object[] args) throws Throwable
    {
       return ReflectionUtils.newInstance(constructor, args);

Modified: projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/introspection/ReflectFieldInfoImpl.java
===================================================================
--- projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/introspection/ReflectFieldInfoImpl.java	2009-03-16 11:39:19 UTC (rev 85913)
+++ projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/introspection/ReflectFieldInfoImpl.java	2009-03-16 11:39:48 UTC (rev 85914)
@@ -125,12 +125,14 @@
       }
    }
 
+   @Override
    public Object get(Object target) throws Throwable
    {
       accessCheck();
       return ReflectionUtils.getField(field, target);
    }
 
+   @Override
    public Object set(Object target, Object value) throws Throwable
    {
       accessCheck();

Modified: projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/introspection/ReflectMethodInfoImpl.java
===================================================================
--- projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/introspection/ReflectMethodInfoImpl.java	2009-03-16 11:39:19 UTC (rev 85913)
+++ projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/introspection/ReflectMethodInfoImpl.java	2009-03-16 11:39:48 UTC (rev 85914)
@@ -145,6 +145,7 @@
       }
    }
 
+   @Override
    public Object invoke(Object target, Object[] args) throws Throwable
    {
       accessCheck();

Modified: projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/javassist/JavassistMethodInfo.java
===================================================================
--- projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/javassist/JavassistMethodInfo.java	2009-03-16 11:39:19 UTC (rev 85913)
+++ projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/javassist/JavassistMethodInfo.java	2009-03-16 11:39:48 UTC (rev 85914)
@@ -21,7 +21,6 @@
 */
 package org.jboss.reflect.plugins.javassist;
 
-import java.lang.reflect.Modifier;
 import java.util.Arrays;
 
 import javassist.CannotCompileException;

Modified: projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/javassist/JavassistTypeInfo.java
===================================================================
--- projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/javassist/JavassistTypeInfo.java	2009-03-16 11:39:19 UTC (rev 85913)
+++ projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/javassist/JavassistTypeInfo.java	2009-03-16 11:39:48 UTC (rev 85914)
@@ -23,7 +23,6 @@
 
 import java.io.IOException;
 import java.lang.reflect.Array;
-import java.lang.reflect.Modifier;
 import java.util.Collection;
 import java.util.Map;
 import java.util.concurrent.ConcurrentHashMap;

Modified: projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/javassist/JavassistTypeInfoFactory.java
===================================================================
--- projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/javassist/JavassistTypeInfoFactory.java	2009-03-16 11:39:19 UTC (rev 85913)
+++ projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/javassist/JavassistTypeInfoFactory.java	2009-03-16 11:39:48 UTC (rev 85914)
@@ -23,8 +23,10 @@
 
 import java.lang.reflect.Type;
 
+import org.jboss.reflect.spi.ClassInfo;
+import org.jboss.reflect.spi.MutableClassInfo;
 import org.jboss.reflect.spi.TypeInfo;
-import org.jboss.reflect.spi.TypeInfoFactory;
+import org.jboss.reflect.spi.MutableTypeInfoFactory;
 
 /**
  * An javassist type factory that uses a static delegate.<p>
@@ -34,12 +36,12 @@
  * 
  * @author <a href="mailto:adrian at jboss.org">Adrian Brock</a>
  */
-public class JavassistTypeInfoFactory implements TypeInfoFactory
+public class JavassistTypeInfoFactory implements MutableTypeInfoFactory
 {
    /** The delegate */
    protected static JavassistTypeInfoFactoryImpl delegate = new JavassistTypeInfoFactoryImpl();
 
-   static TypeInfoFactory getDelegate()
+   static MutableTypeInfoFactory getDelegate()
    {
       return delegate;
    }
@@ -59,4 +61,29 @@
       return delegate.getTypeInfo(type);
    }
 
+   public MutableClassInfo createNewMutableClass(String name)
+   {
+      return delegate.createNewMutableClass(name);
+   }
+
+   public MutableClassInfo createNewMutableClass(String name, ClassInfo superClass)
+   {
+      return delegate.createNewMutableClass(name, superClass);
+   }
+
+   public MutableClassInfo createNewMutableInterface(String name)
+   {
+      return delegate.createNewMutableInterface(name);
+   }
+
+   public MutableClassInfo createNewMutableInterface(String name, ClassInfo superClass)
+   {
+      return delegate.createNewMutableInterface(name, superClass);
+   }
+
+   public MutableClassInfo getMutable(String name, ClassLoader cl)
+   {
+      return delegate.getMutable(name, cl);
+   }
+
 }

Modified: projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/javassist/JavassistTypeInfoFactoryImpl.java
===================================================================
--- projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/javassist/JavassistTypeInfoFactoryImpl.java	2009-03-16 11:39:19 UTC (rev 85913)
+++ projects/jboss-reflect/trunk/src/main/java/org/jboss/reflect/plugins/javassist/JavassistTypeInfoFactoryImpl.java	2009-03-16 11:39:48 UTC (rev 85914)
@@ -281,7 +281,6 @@
     * @param clazz the class
     * @return the info
     */
-   @SuppressWarnings("unchecked")
    public Object get(CtClass clazz)
    {
       if (clazz == null)




More information about the jboss-cvs-commits mailing list