Author: niallp
Date: Tue Aug  4 21:06:52 2009
New Revision: 800975

URL: http://svn.apache.org/viewvc?rev=800975&view=rev
Log:
BEANUTILS-333 Avoid calling setAccessible() if not needed - thanks to Lukasz 
Lenart

Modified:
    
commons/proper/beanutils/trunk/src/java/org/apache/commons/beanutils/MethodUtils.java

Modified: 
commons/proper/beanutils/trunk/src/java/org/apache/commons/beanutils/MethodUtils.java
URL: 
http://svn.apache.org/viewvc/commons/proper/beanutils/trunk/src/java/org/apache/commons/beanutils/MethodUtils.java?rev=800975&r1=800974&r2=800975&view=diff
==============================================================================
--- 
commons/proper/beanutils/trunk/src/java/org/apache/commons/beanutils/MethodUtils.java
 (original)
+++ 
commons/proper/beanutils/trunk/src/java/org/apache/commons/beanutils/MethodUtils.java
 Tue Aug  4 21:06:52 2009
@@ -1060,7 +1060,9 @@
             //
             // Better workarounds would be greatfully accepted.
             //
-            method.setAccessible(true);
+            if (!method.isAccessible()) {
+                method.setAccessible(true);
+            }
             
         } catch (SecurityException se) {
             // log but continue just in case the method.invoke works anyway


Reply via email to