diff options
-rw-r--r-- | ChangeLog | 10 | ||||
-rw-r--r-- | thread.c | 5 | ||||
-rw-r--r-- | thread_pthread.c | 7 | ||||
-rw-r--r-- | thread_win32.c | 11 |
4 files changed, 30 insertions, 3 deletions
@@ -1,3 +1,13 @@ +Mon Nov 2 12:55:50 2009 NAKAMURA Usaku <usa@ruby-lang.org> + + * thread_{pthread,win32}.c (native_stop_timer_thread): join the thread + here. + + * thread_{pthread,win32}.c (native_reset_timer_thread): new function. + + * thread.c (rb_thread_stop_timer_thread, rb_thread_reset_timer_thread): + call above function instead of simply seting 0. + Mon Nov 2 11:22:19 2009 NARUSE, Yui <naruse@ruby-lang.org> * ext/dl/win32/lib/win32/registry.rb: update rdoc. [ruby-core:26022] @@ -2651,15 +2651,14 @@ void rb_thread_stop_timer_thread(void) { if (timer_thread_id && native_stop_timer_thread()) { - native_thread_join(timer_thread_id); - timer_thread_id = 0; + native_reset_timer_thread(); } } void rb_thread_reset_timer_thread(void) { - timer_thread_id = 0; + native_reset_timer_thread(); } void diff --git a/thread_pthread.c b/thread_pthread.c index a4e7f25d75..e3ff6ee714 100644 --- a/thread_pthread.c +++ b/thread_pthread.c @@ -816,9 +816,16 @@ native_stop_timer_thread(void) native_cond_signal(&timer_thread_cond); } native_mutex_unlock(&timer_thread_lock); + native_thread_join(timer_thread_id); return stopped; } +static void +native_reset_timer_thread(void) +{ + timer_thread_id = 0; +} + #ifdef HAVE_SIGALTSTACK int ruby_stack_overflowed_p(const rb_thread_t *th, const void *addr) diff --git a/thread_win32.c b/thread_win32.c index f404a4f312..139b94a94c 100644 --- a/thread_win32.c +++ b/thread_win32.c @@ -564,10 +564,21 @@ native_stop_timer_thread(void) { int stopped = --system_working <= 0; if (stopped) { + SetEvent(timer_thread_lock); + native_thread_join(timer_thread_id); CloseHandle(timer_thread_lock); timer_thread_lock = 0; } return stopped; } +static void +native_reset_timer_thread(void) +{ + if (timer_thread_id) { + CloseHandle(timer_thread_id); + timer_thread_id = 0; + } +} + #endif /* THREAD_SYSTEM_DEPENDENT_IMPLEMENTATION */ |