commit dbe6f4613ae0161b169f4fca8a616b0b393370ab
author Mathieu Chartier <mathieuc@google.com> Tue Sep 25 16:54:50 2012 -0700
committer Ian Rogers <irogers@google.com> Tue Sep 25 22:36:20 2012 -0700
tree bcf21fa802f8ae261522e8c9bab3685020908d13
parent 474b6da273c7ce6df50a4e51eb9929a77e1611c3

Change Thread::peer_ to be a jobject instead of an Object*

Fixes issue where GC was freeing the java peer if the parent thread exited before the child thread got registered.

Change-Id: I6fbe74865d5827d243ac55fc396679afa97ff2f9

src/thread.h

diff --git a/src/thread.h b/src/thread.h
index 34dfdbd..a58fb65 100644
--- a/src/thread.h
+++ b/src/thread.h
@@ -283,7 +283,7 @@
   // Sets the thread's name.
   void SetThreadName(const char* name) SHARED_LOCKS_REQUIRED(Locks::mutator_lock_);
 
-  Object* GetPeer() const SHARED_LOCKS_REQUIRED(Locks::mutator_lock_) {
+  jobject GetPeer() const {
     return peer_;
   }
 
@@ -702,7 +702,7 @@
   Thread* self_;
 
   // Our managed peer (an instance of java.lang.Thread).
-  Object* peer_;
+  jobject peer_;
 
   // The "lowest addressable byte" of the stack
   byte* stack_begin_;