You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by si...@apache.org on 2012/02/05 00:58:15 UTC

svn commit: r1240646 - /commons/sandbox/beanutils2/trunk/src/main/java/org/apache/commons/beanutils2/PropertyDescriptorsRegistry.java

Author: simonetripodi
Date: Sat Feb  4 23:58:13 2012
New Revision: 1240646

URL: http://svn.apache.org/viewvc?rev=1240646&view=rev
Log:
this class is a singleton, methods registry is a singleton, no needs to have the methods registry as a class member

Modified:
    commons/sandbox/beanutils2/trunk/src/main/java/org/apache/commons/beanutils2/PropertyDescriptorsRegistry.java

Modified: commons/sandbox/beanutils2/trunk/src/main/java/org/apache/commons/beanutils2/PropertyDescriptorsRegistry.java
URL: http://svn.apache.org/viewvc/commons/sandbox/beanutils2/trunk/src/main/java/org/apache/commons/beanutils2/PropertyDescriptorsRegistry.java?rev=1240646&r1=1240645&r2=1240646&view=diff
==============================================================================
--- commons/sandbox/beanutils2/trunk/src/main/java/org/apache/commons/beanutils2/PropertyDescriptorsRegistry.java (original)
+++ commons/sandbox/beanutils2/trunk/src/main/java/org/apache/commons/beanutils2/PropertyDescriptorsRegistry.java Sat Feb  4 23:58:13 2012
@@ -40,8 +40,6 @@ import java.util.concurrent.locks.Reentr
 final class PropertyDescriptorsRegistry
 {
 
-    private static final AccessibleObjectsRegistry<Method> METHODS_REGISTRY = AccessibleObjectsRegistry.getMethodsRegistry();
-
     private static final PropertyDescriptorsRegistry INSTANCE = new PropertyDescriptorsRegistry();
 
     public static PropertyDescriptorsRegistry getInstance()
@@ -49,6 +47,8 @@ final class PropertyDescriptorsRegistry
         return INSTANCE;
     }
 
+    private final AccessibleObjectsRegistry<Method> methodsRegistry = AccessibleObjectsRegistry.getMethodsRegistry();
+
     private final ReadWriteLock lock = new ReentrantReadWriteLock();
 
     private final Map<Class<?>, WeakReference<Map<String, PropertyDescriptor>>> cache =
@@ -103,7 +103,7 @@ final class PropertyDescriptorsRegistry
         }
     }
 
-    private static void makeMethodsAccessible( Class<?> beanType, PropertyDescriptor propertyDescriptor )
+    private void makeMethodsAccessible( Class<?> beanType, PropertyDescriptor propertyDescriptor )
         throws IntrospectionException
     {
         // we need to make sure that methods are accessible for anonymous types
@@ -111,13 +111,13 @@ final class PropertyDescriptorsRegistry
         if ( propertyDescriptor.getReadMethod() != null )
         {
             Method readMethod = propertyDescriptor.getReadMethod();
-            readMethod = METHODS_REGISTRY.get( true, beanType, readMethod.getName() );
+            readMethod = methodsRegistry.get( true, beanType, readMethod.getName() );
             propertyDescriptor.setReadMethod( readMethod );
         }
         if ( propertyDescriptor.getWriteMethod() != null )
         {
             Method writeMethod = propertyDescriptor.getWriteMethod();
-            writeMethod = METHODS_REGISTRY.get( true, beanType, writeMethod.getName(), writeMethod.getParameterTypes() );
+            writeMethod = methodsRegistry.get( true, beanType, writeMethod.getName(), writeMethod.getParameterTypes() );
             propertyDescriptor.setWriteMethod( writeMethod );
         }
     }



Re: svn commit: r1240646 - /commons/sandbox/beanutils2/trunk/src/main/java/org/apache/commons/beanutils2/PropertyDescriptorsRegistry.java

Posted by Simone Tripodi <si...@apache.org>.
the subject of my tense is the method, not the class, I didn't mean to
put a static reference - but next time I'll write more explicitly so
will be easier to avoid misunderstanding.

http://people.apache.org/~simonetripodi/
http://simonetripodi.livejournal.com/
http://twitter.com/simonetripodi
http://www.99soft.org/



On Sun, Feb 5, 2012 at 2:23 PM, Benedikt Ritter
<be...@systemoutprintln.de> wrote:
> That's what I was saying ;-) You suggested a static reference in Jira [1]
> and that confused me a bit. So I like your change, because that is kind of
> what I did in the first place.
>
> [1]
> https://issues.apache.org/jira/browse/SANDBOX-379?focusedCommentId=13198666&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-13198666
>
> Am 05.02.2012 13:26, schrieb Simone Tripodi:
>
>> that's because you added the registry as static reference in your
>> previous patch, just changed the logic.
>>
>> alles gute!
>> -Simo
>>
>> http://people.apache.org/~simonetripodi/
>> http://simonetripodi.livejournal.com/
>> http://twitter.com/simonetripodi
>> http://www.99soft.org/
>>
>>
>>
>> On Sun, Feb 5, 2012 at 12:20 PM, Benedikt Ritter
>> <be...@systemoutprintln.de>  wrote:
>>>
>>> Thanks for that patch. I was a bit confused, when you suggested to access
>>> MethodsRegistry statically in JIRA. Now that seems to be a nice solution.
>>>
>>> I'm currently working on populate() and there are a few things worth
>>> discussing. I hope that I have the time to write something down this
>>> evening.
>>>
>>> Have a nice sunday!
>>> Benedikt
>>>
>>> Am 05.02.2012 00:58, schrieb simonetripodi@apache.org:
>>>>
>>>>
>>>> Author: simonetripodi
>>>> Date: Sat Feb  4 23:58:13 2012
>>>> New Revision: 1240646
>>>>
>>>> URL: http://svn.apache.org/viewvc?rev=1240646&view=rev
>>>> Log:
>>>> this class is a singleton, methods registry is a singleton, no needs to
>>>> have the methods registry as a class member
>>>>
>>>> Modified:
>>>>
>>>>
>>>> commons/sandbox/beanutils2/trunk/src/main/java/org/apache/commons/beanutils2/PropertyDescriptorsRegistry.java
>>>>
>>>> Modified:
>>>>
>>>> commons/sandbox/beanutils2/trunk/src/main/java/org/apache/commons/beanutils2/PropertyDescriptorsRegistry.java
>>>> URL:
>>>>
>>>> http://svn.apache.org/viewvc/commons/sandbox/beanutils2/trunk/src/main/java/org/apache/commons/beanutils2/PropertyDescriptorsRegistry.java?rev=1240646&r1=1240645&r2=1240646&view=diff
>>>>
>>>>
>>>> ==============================================================================
>>>> ---
>>>>
>>>> commons/sandbox/beanutils2/trunk/src/main/java/org/apache/commons/beanutils2/PropertyDescriptorsRegistry.java
>>>> (original)
>>>> +++
>>>>
>>>> commons/sandbox/beanutils2/trunk/src/main/java/org/apache/commons/beanutils2/PropertyDescriptorsRegistry.java
>>>> Sat Feb  4 23:58:13 2012
>>>> @@ -40,8 +40,6 @@ import java.util.concurrent.locks.Reentr
>>>>  final class PropertyDescriptorsRegistry
>>>>  {
>>>>
>>>> -    private static final AccessibleObjectsRegistry<Method>
>>>>  METHODS_REGISTRY = AccessibleObjectsRegistry.getMethodsRegistry();
>>>> -
>>>>      private static final PropertyDescriptorsRegistry INSTANCE = new
>>>> PropertyDescriptorsRegistry();
>>>>
>>>>      public static PropertyDescriptorsRegistry getInstance()
>>>> @@ -49,6 +47,8 @@ final class PropertyDescriptorsRegistry
>>>>          return INSTANCE;
>>>>      }
>>>>
>>>> +    private final AccessibleObjectsRegistry<Method>    methodsRegistry
>>>> =
>>>> AccessibleObjectsRegistry.getMethodsRegistry();
>>>> +
>>>>      private final ReadWriteLock lock = new ReentrantReadWriteLock();
>>>>
>>>>      private final Map<Class<?>, WeakReference<Map<String,
>>>> PropertyDescriptor>>>    cache =
>>>> @@ -103,7 +103,7 @@ final class PropertyDescriptorsRegistry
>>>>          }
>>>>      }
>>>>
>>>> -    private static void makeMethodsAccessible( Class<?>    beanType,
>>>> PropertyDescriptor propertyDescriptor )
>>>> +    private void makeMethodsAccessible( Class<?>    beanType,
>>>> PropertyDescriptor propertyDescriptor )
>>>>          throws IntrospectionException
>>>>      {
>>>>          // we need to make sure that methods are accessible for
>>>> anonymous
>>>> types
>>>> @@ -111,13 +111,13 @@ final class PropertyDescriptorsRegistry
>>>>          if ( propertyDescriptor.getReadMethod() != null )
>>>>          {
>>>>              Method readMethod = propertyDescriptor.getReadMethod();
>>>> -            readMethod = METHODS_REGISTRY.get( true, beanType,
>>>> readMethod.getName() );
>>>> +            readMethod = methodsRegistry.get( true, beanType,
>>>> readMethod.getName() );
>>>>              propertyDescriptor.setReadMethod( readMethod );
>>>>          }
>>>>          if ( propertyDescriptor.getWriteMethod() != null )
>>>>          {
>>>>              Method writeMethod = propertyDescriptor.getWriteMethod();
>>>> -            writeMethod = METHODS_REGISTRY.get( true, beanType,
>>>> writeMethod.getName(), writeMethod.getParameterTypes() );
>>>> +            writeMethod = methodsRegistry.get( true, beanType,
>>>> writeMethod.getName(), writeMethod.getParameterTypes() );
>>>>              propertyDescriptor.setWriteMethod( writeMethod );
>>>>          }
>>>>      }
>>>>
>>>
>>>
>>> ---------------------------------------------------------------------
>>> To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
>>> For additional commands, e-mail: dev-help@commons.apache.org
>>>
>>
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
>> For additional commands, e-mail: dev-help@commons.apache.org
>
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
> For additional commands, e-mail: dev-help@commons.apache.org
>

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org


Re: svn commit: r1240646 - /commons/sandbox/beanutils2/trunk/src/main/java/org/apache/commons/beanutils2/PropertyDescriptorsRegistry.java

Posted by Benedikt Ritter <be...@systemoutprintln.de>.
That's what I was saying ;-) You suggested a static reference in Jira 
[1] and that confused me a bit. So I like your change, because that is 
kind of what I did in the first place.

[1] 
https://issues.apache.org/jira/browse/SANDBOX-379?focusedCommentId=13198666&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-13198666

Am 05.02.2012 13:26, schrieb Simone Tripodi:
> that's because you added the registry as static reference in your
> previous patch, just changed the logic.
>
> alles gute!
> -Simo
>
> http://people.apache.org/~simonetripodi/
> http://simonetripodi.livejournal.com/
> http://twitter.com/simonetripodi
> http://www.99soft.org/
>
>
>
> On Sun, Feb 5, 2012 at 12:20 PM, Benedikt Ritter
> <be...@systemoutprintln.de>  wrote:
>> Thanks for that patch. I was a bit confused, when you suggested to access
>> MethodsRegistry statically in JIRA. Now that seems to be a nice solution.
>>
>> I'm currently working on populate() and there are a few things worth
>> discussing. I hope that I have the time to write something down this
>> evening.
>>
>> Have a nice sunday!
>> Benedikt
>>
>> Am 05.02.2012 00:58, schrieb simonetripodi@apache.org:
>>>
>>> Author: simonetripodi
>>> Date: Sat Feb  4 23:58:13 2012
>>> New Revision: 1240646
>>>
>>> URL: http://svn.apache.org/viewvc?rev=1240646&view=rev
>>> Log:
>>> this class is a singleton, methods registry is a singleton, no needs to
>>> have the methods registry as a class member
>>>
>>> Modified:
>>>
>>> commons/sandbox/beanutils2/trunk/src/main/java/org/apache/commons/beanutils2/PropertyDescriptorsRegistry.java
>>>
>>> Modified:
>>> commons/sandbox/beanutils2/trunk/src/main/java/org/apache/commons/beanutils2/PropertyDescriptorsRegistry.java
>>> URL:
>>> http://svn.apache.org/viewvc/commons/sandbox/beanutils2/trunk/src/main/java/org/apache/commons/beanutils2/PropertyDescriptorsRegistry.java?rev=1240646&r1=1240645&r2=1240646&view=diff
>>>
>>> ==============================================================================
>>> ---
>>> commons/sandbox/beanutils2/trunk/src/main/java/org/apache/commons/beanutils2/PropertyDescriptorsRegistry.java
>>> (original)
>>> +++
>>> commons/sandbox/beanutils2/trunk/src/main/java/org/apache/commons/beanutils2/PropertyDescriptorsRegistry.java
>>> Sat Feb  4 23:58:13 2012
>>> @@ -40,8 +40,6 @@ import java.util.concurrent.locks.Reentr
>>>   final class PropertyDescriptorsRegistry
>>>   {
>>>
>>> -    private static final AccessibleObjectsRegistry<Method>
>>>   METHODS_REGISTRY = AccessibleObjectsRegistry.getMethodsRegistry();
>>> -
>>>       private static final PropertyDescriptorsRegistry INSTANCE = new
>>> PropertyDescriptorsRegistry();
>>>
>>>       public static PropertyDescriptorsRegistry getInstance()
>>> @@ -49,6 +47,8 @@ final class PropertyDescriptorsRegistry
>>>           return INSTANCE;
>>>       }
>>>
>>> +    private final AccessibleObjectsRegistry<Method>    methodsRegistry =
>>> AccessibleObjectsRegistry.getMethodsRegistry();
>>> +
>>>       private final ReadWriteLock lock = new ReentrantReadWriteLock();
>>>
>>>       private final Map<Class<?>, WeakReference<Map<String,
>>> PropertyDescriptor>>>    cache =
>>> @@ -103,7 +103,7 @@ final class PropertyDescriptorsRegistry
>>>           }
>>>       }
>>>
>>> -    private static void makeMethodsAccessible( Class<?>    beanType,
>>> PropertyDescriptor propertyDescriptor )
>>> +    private void makeMethodsAccessible( Class<?>    beanType,
>>> PropertyDescriptor propertyDescriptor )
>>>           throws IntrospectionException
>>>       {
>>>           // we need to make sure that methods are accessible for anonymous
>>> types
>>> @@ -111,13 +111,13 @@ final class PropertyDescriptorsRegistry
>>>           if ( propertyDescriptor.getReadMethod() != null )
>>>           {
>>>               Method readMethod = propertyDescriptor.getReadMethod();
>>> -            readMethod = METHODS_REGISTRY.get( true, beanType,
>>> readMethod.getName() );
>>> +            readMethod = methodsRegistry.get( true, beanType,
>>> readMethod.getName() );
>>>               propertyDescriptor.setReadMethod( readMethod );
>>>           }
>>>           if ( propertyDescriptor.getWriteMethod() != null )
>>>           {
>>>               Method writeMethod = propertyDescriptor.getWriteMethod();
>>> -            writeMethod = METHODS_REGISTRY.get( true, beanType,
>>> writeMethod.getName(), writeMethod.getParameterTypes() );
>>> +            writeMethod = methodsRegistry.get( true, beanType,
>>> writeMethod.getName(), writeMethod.getParameterTypes() );
>>>               propertyDescriptor.setWriteMethod( writeMethod );
>>>           }
>>>       }
>>>
>>
>>
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
>> For additional commands, e-mail: dev-help@commons.apache.org
>>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
> For additional commands, e-mail: dev-help@commons.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org


Re: svn commit: r1240646 - /commons/sandbox/beanutils2/trunk/src/main/java/org/apache/commons/beanutils2/PropertyDescriptorsRegistry.java

Posted by Simone Tripodi <si...@apache.org>.
that's because you added the registry as static reference in your
previous patch, just changed the logic.

alles gute!
-Simo

http://people.apache.org/~simonetripodi/
http://simonetripodi.livejournal.com/
http://twitter.com/simonetripodi
http://www.99soft.org/



On Sun, Feb 5, 2012 at 12:20 PM, Benedikt Ritter
<be...@systemoutprintln.de> wrote:
> Thanks for that patch. I was a bit confused, when you suggested to access
> MethodsRegistry statically in JIRA. Now that seems to be a nice solution.
>
> I'm currently working on populate() and there are a few things worth
> discussing. I hope that I have the time to write something down this
> evening.
>
> Have a nice sunday!
> Benedikt
>
> Am 05.02.2012 00:58, schrieb simonetripodi@apache.org:
>>
>> Author: simonetripodi
>> Date: Sat Feb  4 23:58:13 2012
>> New Revision: 1240646
>>
>> URL: http://svn.apache.org/viewvc?rev=1240646&view=rev
>> Log:
>> this class is a singleton, methods registry is a singleton, no needs to
>> have the methods registry as a class member
>>
>> Modified:
>>
>> commons/sandbox/beanutils2/trunk/src/main/java/org/apache/commons/beanutils2/PropertyDescriptorsRegistry.java
>>
>> Modified:
>> commons/sandbox/beanutils2/trunk/src/main/java/org/apache/commons/beanutils2/PropertyDescriptorsRegistry.java
>> URL:
>> http://svn.apache.org/viewvc/commons/sandbox/beanutils2/trunk/src/main/java/org/apache/commons/beanutils2/PropertyDescriptorsRegistry.java?rev=1240646&r1=1240645&r2=1240646&view=diff
>>
>> ==============================================================================
>> ---
>> commons/sandbox/beanutils2/trunk/src/main/java/org/apache/commons/beanutils2/PropertyDescriptorsRegistry.java
>> (original)
>> +++
>> commons/sandbox/beanutils2/trunk/src/main/java/org/apache/commons/beanutils2/PropertyDescriptorsRegistry.java
>> Sat Feb  4 23:58:13 2012
>> @@ -40,8 +40,6 @@ import java.util.concurrent.locks.Reentr
>>  final class PropertyDescriptorsRegistry
>>  {
>>
>> -    private static final AccessibleObjectsRegistry<Method>
>>  METHODS_REGISTRY = AccessibleObjectsRegistry.getMethodsRegistry();
>> -
>>      private static final PropertyDescriptorsRegistry INSTANCE = new
>> PropertyDescriptorsRegistry();
>>
>>      public static PropertyDescriptorsRegistry getInstance()
>> @@ -49,6 +47,8 @@ final class PropertyDescriptorsRegistry
>>          return INSTANCE;
>>      }
>>
>> +    private final AccessibleObjectsRegistry<Method>  methodsRegistry =
>> AccessibleObjectsRegistry.getMethodsRegistry();
>> +
>>      private final ReadWriteLock lock = new ReentrantReadWriteLock();
>>
>>      private final Map<Class<?>, WeakReference<Map<String,
>> PropertyDescriptor>>>  cache =
>> @@ -103,7 +103,7 @@ final class PropertyDescriptorsRegistry
>>          }
>>      }
>>
>> -    private static void makeMethodsAccessible( Class<?>  beanType,
>> PropertyDescriptor propertyDescriptor )
>> +    private void makeMethodsAccessible( Class<?>  beanType,
>> PropertyDescriptor propertyDescriptor )
>>          throws IntrospectionException
>>      {
>>          // we need to make sure that methods are accessible for anonymous
>> types
>> @@ -111,13 +111,13 @@ final class PropertyDescriptorsRegistry
>>          if ( propertyDescriptor.getReadMethod() != null )
>>          {
>>              Method readMethod = propertyDescriptor.getReadMethod();
>> -            readMethod = METHODS_REGISTRY.get( true, beanType,
>> readMethod.getName() );
>> +            readMethod = methodsRegistry.get( true, beanType,
>> readMethod.getName() );
>>              propertyDescriptor.setReadMethod( readMethod );
>>          }
>>          if ( propertyDescriptor.getWriteMethod() != null )
>>          {
>>              Method writeMethod = propertyDescriptor.getWriteMethod();
>> -            writeMethod = METHODS_REGISTRY.get( true, beanType,
>> writeMethod.getName(), writeMethod.getParameterTypes() );
>> +            writeMethod = methodsRegistry.get( true, beanType,
>> writeMethod.getName(), writeMethod.getParameterTypes() );
>>              propertyDescriptor.setWriteMethod( writeMethod );
>>          }
>>      }
>>
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
> For additional commands, e-mail: dev-help@commons.apache.org
>

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org


Re: svn commit: r1240646 - /commons/sandbox/beanutils2/trunk/src/main/java/org/apache/commons/beanutils2/PropertyDescriptorsRegistry.java

Posted by Benedikt Ritter <be...@systemoutprintln.de>.
Thanks for that patch. I was a bit confused, when you suggested to 
access MethodsRegistry statically in JIRA. Now that seems to be a nice 
solution.

I'm currently working on populate() and there are a few things worth 
discussing. I hope that I have the time to write something down this 
evening.

Have a nice sunday!
Benedikt

Am 05.02.2012 00:58, schrieb simonetripodi@apache.org:
> Author: simonetripodi
> Date: Sat Feb  4 23:58:13 2012
> New Revision: 1240646
>
> URL: http://svn.apache.org/viewvc?rev=1240646&view=rev
> Log:
> this class is a singleton, methods registry is a singleton, no needs to have the methods registry as a class member
>
> Modified:
>      commons/sandbox/beanutils2/trunk/src/main/java/org/apache/commons/beanutils2/PropertyDescriptorsRegistry.java
>
> Modified: commons/sandbox/beanutils2/trunk/src/main/java/org/apache/commons/beanutils2/PropertyDescriptorsRegistry.java
> URL: http://svn.apache.org/viewvc/commons/sandbox/beanutils2/trunk/src/main/java/org/apache/commons/beanutils2/PropertyDescriptorsRegistry.java?rev=1240646&r1=1240645&r2=1240646&view=diff
> ==============================================================================
> --- commons/sandbox/beanutils2/trunk/src/main/java/org/apache/commons/beanutils2/PropertyDescriptorsRegistry.java (original)
> +++ commons/sandbox/beanutils2/trunk/src/main/java/org/apache/commons/beanutils2/PropertyDescriptorsRegistry.java Sat Feb  4 23:58:13 2012
> @@ -40,8 +40,6 @@ import java.util.concurrent.locks.Reentr
>   final class PropertyDescriptorsRegistry
>   {
>
> -    private static final AccessibleObjectsRegistry<Method>  METHODS_REGISTRY = AccessibleObjectsRegistry.getMethodsRegistry();
> -
>       private static final PropertyDescriptorsRegistry INSTANCE = new PropertyDescriptorsRegistry();
>
>       public static PropertyDescriptorsRegistry getInstance()
> @@ -49,6 +47,8 @@ final class PropertyDescriptorsRegistry
>           return INSTANCE;
>       }
>
> +    private final AccessibleObjectsRegistry<Method>  methodsRegistry = AccessibleObjectsRegistry.getMethodsRegistry();
> +
>       private final ReadWriteLock lock = new ReentrantReadWriteLock();
>
>       private final Map<Class<?>, WeakReference<Map<String, PropertyDescriptor>>>  cache =
> @@ -103,7 +103,7 @@ final class PropertyDescriptorsRegistry
>           }
>       }
>
> -    private static void makeMethodsAccessible( Class<?>  beanType, PropertyDescriptor propertyDescriptor )
> +    private void makeMethodsAccessible( Class<?>  beanType, PropertyDescriptor propertyDescriptor )
>           throws IntrospectionException
>       {
>           // we need to make sure that methods are accessible for anonymous types
> @@ -111,13 +111,13 @@ final class PropertyDescriptorsRegistry
>           if ( propertyDescriptor.getReadMethod() != null )
>           {
>               Method readMethod = propertyDescriptor.getReadMethod();
> -            readMethod = METHODS_REGISTRY.get( true, beanType, readMethod.getName() );
> +            readMethod = methodsRegistry.get( true, beanType, readMethod.getName() );
>               propertyDescriptor.setReadMethod( readMethod );
>           }
>           if ( propertyDescriptor.getWriteMethod() != null )
>           {
>               Method writeMethod = propertyDescriptor.getWriteMethod();
> -            writeMethod = METHODS_REGISTRY.get( true, beanType, writeMethod.getName(), writeMethod.getParameterTypes() );
> +            writeMethod = methodsRegistry.get( true, beanType, writeMethod.getName(), writeMethod.getParameterTypes() );
>               propertyDescriptor.setWriteMethod( writeMethod );
>           }
>       }
>


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org