You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tomee.apache.org by rm...@apache.org on 2012/02/22 15:52:19 UTC
svn commit: r1292325 -
/openejb/trunk/openejb/container/openejb-core/src/main/java/org/apache/openejb/util/reflection/ReflectionUtil.java
Author: rmannibucau
Date: Wed Feb 22 14:52:19 2012
New Revision: 1292325
URL: http://svn.apache.org/viewvc?rev=1292325&view=rev
Log:
if we don't catch the NoSuchFieldException the loop is useless
Modified:
openejb/trunk/openejb/container/openejb-core/src/main/java/org/apache/openejb/util/reflection/ReflectionUtil.java
Modified: openejb/trunk/openejb/container/openejb-core/src/main/java/org/apache/openejb/util/reflection/ReflectionUtil.java
URL: http://svn.apache.org/viewvc/openejb/trunk/openejb/container/openejb-core/src/main/java/org/apache/openejb/util/reflection/ReflectionUtil.java?rev=1292325&r1=1292324&r2=1292325&view=diff
==============================================================================
--- openejb/trunk/openejb/container/openejb-core/src/main/java/org/apache/openejb/util/reflection/ReflectionUtil.java (original)
+++ openejb/trunk/openejb/container/openejb-core/src/main/java/org/apache/openejb/util/reflection/ReflectionUtil.java Wed Feb 22 14:52:19 2012
@@ -32,7 +32,11 @@ public final class ReflectionUtil {
try {
Class<?> current = notNullInstance.getClass();
while (!current.equals(Object.class) && field == null) {
- field = current.getDeclaredField(fieldName);
+ try {
+ field = current.getDeclaredField(fieldName);
+ } catch (NoSuchFieldException nsfe) {
+ // ignored, will try the same at next level
+ }
current = current.getSuperclass();
}
accessible = field.isAccessible();