You are viewing a plain text version of this content. The canonical link for it is here.

Posted to scm@geronimo.apache.org by xi...@apache.org on 2013/05/23 04:38:18 UTC

svn commit: r1485546 - /geronimo/tck/tags/3.0.1/geronimo-porting/src/main/java/org/apache/geronimo/test/tck/BeansImpl.java

Author: xiaming
Date: Thu May 23 02:38:18 2013
New Revision: 1485546

URL: http://svn.apache.org/r1485546
Log:
Port the fix for GERONIMO-6034 regression into the tag

Modified:
    geronimo/tck/tags/3.0.1/geronimo-porting/src/main/java/org/apache/geronimo/test/tck/BeansImpl.java

Modified: geronimo/tck/tags/3.0.1/geronimo-porting/src/main/java/org/apache/geronimo/test/tck/BeansImpl.java
URL: http://svn.apache.org/viewvc/geronimo/tck/tags/3.0.1/geronimo-porting/src/main/java/org/apache/geronimo/test/tck/BeansImpl.java?rev=1485546&r1=1485545&r2=1485546&view=diff
==============================================================================
--- geronimo/tck/tags/3.0.1/geronimo-porting/src/main/java/org/apache/geronimo/test/tck/BeansImpl.java (original)
+++ geronimo/tck/tags/3.0.1/geronimo-porting/src/main/java/org/apache/geronimo/test/tck/BeansImpl.java Thu May 23 02:38:18 2013
@@ -34,7 +34,12 @@ public class BeansImpl implements Beans 
     public boolean isProxy(Object instance) {
         System.out.println("isProxy: " + instance);
         final String name = instance.getClass().getName();
-        return name.contains("$$") || name.startsWith("$Proxy");
+        if (name.contains("$$")) {
+            return true;
+        } else {
+            int pos = name.lastIndexOf('.');
+            return name.startsWith("$Proxy", pos + 1);
+        }
     }
 
     @Override