Increase use of ScopedJniThreadState.
Move the routines for changing Object* to jobject and vice-versa
(AddLocalReference and Decode) to ScopedJniThreadState to enforce use of
Object*s in the Runnable thread state. In the Runnable thread state
suspension is necessary before GC can take place.
Reduce use of const ClassLoader* as the code bottoms out in FindClass
and with a field assignment where the const is cast away (ie if we're
not going to enforce the const-ness we shouldn't pretend it is).
Refactor the Thread::Attach API so that we're not handling raw Objects on
unattached threads.
Remove some unreachable code.
Change-Id: I0fa969f49ee6a8f10752af74a6b0e04d46b4cd97
diff --git a/src/exception_test.cc b/src/exception_test.cc
index 90bcb7c..269822a 100644
--- a/src/exception_test.cc
+++ b/src/exception_test.cc
@@ -19,6 +19,7 @@
#include "dex_file.h"
#include "gtest/gtest.h"
#include "runtime.h"
+#include "scoped_jni_thread_state.h"
#include "thread.h"
#include "UniquePtr.h"
@@ -160,12 +161,13 @@
#endif
JNIEnv* env = thread->GetJniEnv();
- jobject internal = thread->CreateInternalStackTrace(env);
+ ScopedJniThreadState ts(env);
+ jobject internal = thread->CreateInternalStackTrace(ts);
ASSERT_TRUE(internal != NULL);
jobjectArray ste_array = Thread::InternalStackTraceToStackTraceElementArray(env, internal);
ASSERT_TRUE(ste_array != NULL);
ObjectArray<StackTraceElement>* trace_array =
- Decode<ObjectArray<StackTraceElement>*>(env, ste_array);
+ ts.Decode<ObjectArray<StackTraceElement>*>(ste_array);
ASSERT_TRUE(trace_array != NULL);
ASSERT_TRUE(trace_array->Get(0) != NULL);