* ext/thread/thread.c (unlock_mutex_inner): should adjust join

  target here but not in wake_one.  a patch from John Carter at
  [ruby-core:23490].


git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/branches/ruby_1_8@23482 b2dd03c8-39d4-4d8f-98ff-823fe69b080e

1 files changed, 5 insertions, 14 deletions

diff --git a/ext/thread/thread.c b/ext/thread/thread.c
index b977fa77cf..eed74baeb8 100644
--- a/ext/thread/thread.c
+++ b/ext/thread/thread.c
@@ -231,7 +231,7 @@ run_thread(VALUE thread)
 }
 
 static VALUE
-wake_first(List *list)
+wake_one(List *list)
 {
     VALUE waking;
 
@@ -244,22 +244,10 @@ wake_first(List *list)
 }
 
 static VALUE
-wake_one(List *list)
-{
-    VALUE waking = wake_first(list);
-
-    if (!NIL_P(waking)) {
-	adjust_join(list, waking);
-    }
-
-    return waking;
-}
-
-static VALUE
 wake_all(List *list)
 {
     while (list->entries) {
-        wake_first(list);
+        wake_one(list);
     }
     return Qnil;
 }
@@ -493,6 +481,9 @@ unlock_mutex_inner(Mutex *mutex)
     }
 
     waking = wake_one(&mutex->waiting);
+    if (!NIL_P(waking)) {
+	adjust_join(list, waking);
+    }
     mutex->owner = waking;
 
     return waking;