If JNIEnvExt is-a JNIEnv, we can lose a bunch of reinterpret_casts.
Likewise with JavaVM.
Change-Id: I505e42e9197951b6c82e6dbf8dcc1fc4dd5aad31
diff --git a/src/thread.cc b/src/thread.cc
index d730442..477b149 100644
--- a/src/thread.cc
+++ b/src/thread.cc
@@ -178,7 +178,7 @@
JavaVMExt* vm = runtime->GetJavaVM();
CHECK(vm != NULL);
bool check_jni = vm->check_jni;
- thread->jni_env_ = reinterpret_cast<JNIEnv*>(new JNIEnvExt(thread, check_jni));
+ thread->jni_env_ = new JNIEnvExt(thread, check_jni);
return thread;
}
@@ -240,8 +240,7 @@
switch (kind) {
case kLocal:
{
- JNIEnvExt* env = reinterpret_cast<JNIEnvExt*>(jni_env_);
- IndirectReferenceTable& locals = env->locals;
+ IndirectReferenceTable& locals = jni_env_->locals;
result = locals.Get(ref);
break;
}