You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@openjpa.apache.org by pp...@apache.org on 2007/02/20 19:08:22 UTC

svn commit: r509691 - /incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/DetachedStateManager.java

Author: ppoddar
Date: Tue Feb 20 10:08:22 2007
New Revision: 509691

URL: http://svn.apache.org/viewvc?view=rev&rev=509691
Log:
Use safe and standard way to print a pc.toString() using Exceptions.toString(Object) when logging messages that may cause infinite loop.

Modified:
    incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/DetachedStateManager.java

Modified: incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/DetachedStateManager.java
URL: http://svn.apache.org/viewvc/incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/DetachedStateManager.java?view=diff&rev=509691&r1=509690&r2=509691
==============================================================================
--- incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/DetachedStateManager.java (original)
+++ incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/DetachedStateManager.java Tue Feb 20 10:08:22 2007
@@ -30,6 +30,7 @@
 import org.apache.openjpa.meta.FieldMetaData;
 import org.apache.openjpa.meta.JavaTypes;
 import org.apache.openjpa.meta.ValueMetaData;
+import org.apache.openjpa.util.Exceptions;
 import org.apache.openjpa.util.Proxy;
 import org.apache.openjpa.util.UnsupportedException;
 
@@ -399,7 +400,7 @@
         	// do not access the pc fields by implictly invoking _pc.toString()
         	// may cause infinite loop if again tries to access unloaded field 
             throw new IllegalStateException(_loc.get("unloaded-detached",
-                _pc.getClass()+"@"+System.identityHashCode(_pc)).getMessage());
+               Exceptions.toString(_pc)).getMessage());
     }
 
     public boolean serializing() {