You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by ni...@apache.org on 2009/08/04 23:06:53 UTC
svn commit: r800975 -
/commons/proper/beanutils/trunk/src/java/org/apache/commons/beanutils/MethodUtils.java
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