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 2014/04/09 19:31:09 UTC

svn commit: r1586048 - /tomee/tomee/trunk/container/openejb-core/src/main/java/org/apache/openejb/core/ivm/BaseEjbProxyHandler.java

Author: rmannibucau
Date: Wed Apr  9 17:31:08 2014
New Revision: 1586048

URL: http://svn.apache.org/r1586048
Log:
TOMEE-1175 fixing potential perf issue in equals of BaseEjbProxyHandler

Modified:
    tomee/tomee/trunk/container/openejb-core/src/main/java/org/apache/openejb/core/ivm/BaseEjbProxyHandler.java

Modified: tomee/tomee/trunk/container/openejb-core/src/main/java/org/apache/openejb/core/ivm/BaseEjbProxyHandler.java
URL: http://svn.apache.org/viewvc/tomee/tomee/trunk/container/openejb-core/src/main/java/org/apache/openejb/core/ivm/BaseEjbProxyHandler.java?rev=1586048&r1=1586047&r2=1586048&view=diff
==============================================================================
--- tomee/tomee/trunk/container/openejb-core/src/main/java/org/apache/openejb/core/ivm/BaseEjbProxyHandler.java (original)
+++ tomee/tomee/trunk/container/openejb-core/src/main/java/org/apache/openejb/core/ivm/BaseEjbProxyHandler.java Wed Apr  9 17:31:08 2014
@@ -480,10 +480,12 @@ public abstract class BaseEjbProxyHandle
         if (obj == null) {
             return false;
         }
-        try {
-            obj = ProxyManager.getInvocationHandler(obj);
-        } catch (final IllegalArgumentException e) {
-            return false;
+        if (!BaseEjbProxyHandler.class.isInstance(obj)) {
+            try {
+                obj = ProxyManager.getInvocationHandler(obj);
+            } catch (final IllegalArgumentException e) {
+                return false;
+            }
         }
         if (this == obj) {
             return true;