You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@harmony.apache.org by gs...@apache.org on 2008/01/09 14:50:06 UTC

svn commit: r610387 - /harmony/enhanced/drlvm/trunk/vm/vmcore/src/jni/jni.cpp

Author: gshimansky
Date: Wed Jan  9 05:49:22 2008
New Revision: 610387

URL: http://svn.apache.org/viewvc?rev=610387&view=rev
Log:
Fixed failing assertion due to wrong suspension state in tracing


Modified:
    harmony/enhanced/drlvm/trunk/vm/vmcore/src/jni/jni.cpp

Modified: harmony/enhanced/drlvm/trunk/vm/vmcore/src/jni/jni.cpp
URL: http://svn.apache.org/viewvc/harmony/enhanced/drlvm/trunk/vm/vmcore/src/jni/jni.cpp?rev=610387&r1=610386&r2=610387&view=diff
==============================================================================
--- harmony/enhanced/drlvm/trunk/vm/vmcore/src/jni/jni.cpp (original)
+++ harmony/enhanced/drlvm/trunk/vm/vmcore/src/jni/jni.cpp Wed Jan  9 05:49:22 2008
@@ -882,9 +882,15 @@
 
     if (NULL != new_ref) {
         new_ref->object = ref->object;
+        tmn_suspend_enable();
+        // Gregory -
+        // The following TRACE2 requires suspend to be enabled!
         TRACE2("jni", "NewLocalRef class = " << jobject_to_struct_Class(new_ref));
     }
-    tmn_suspend_enable();   
+    else
+    {
+        tmn_suspend_enable();
+    }
     return new_ref;
 } //NewLocalRef