diff options
author | nobu <nobu@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2007-08-18 06:59:54 +0000 |
---|---|---|
committer | nobu <nobu@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2007-08-18 06:59:54 +0000 |
commit | 6e0c6ac3be8069e939748b00febcf4411ce905cb (patch) | |
tree | 5ee4de1ec435e4d2d12521a2629ed7ace9bb0bac /process.c | |
parent | bade43f9d16d1a03c29077c0f1c8fbf07e19acd1 (diff) |
* process.c (detach_process_watcher): should not pass the pointer
to an auto variable to the thread to be created. pointed and
fix by KUBO Takehiro <kubo at jiubao.org> [ruby-dev:30618]
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@13101 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
Diffstat (limited to 'process.c')
-rw-r--r-- | process.c | 16 |
1 files changed, 7 insertions, 9 deletions
@@ -860,22 +860,21 @@ proc_waitall(void) } static VALUE -detach_process_watcher(int *pid_p) +detach_process_watcher(void *arg) { - rb_pid_t cpid; + rb_pid_t cpid, pid = (rb_pid_t)arg; int status; - for (;;) { - cpid = rb_waitpid(*pid_p, &status, WNOHANG); - if (cpid != 0) return rb_last_status_get(); - rb_thread_sleep(1); + while ((cpid = rb_waitpid(pid, &status, 0)) == 0) { + /* wait while alive */ } + return rb_last_status_get(); } VALUE rb_detach_process(rb_pid_t pid) { - return rb_thread_create(detach_process_watcher, (void*)&pid); + return rb_thread_create(detach_process_watcher, (void*)pid); } @@ -891,8 +890,7 @@ rb_detach_process(rb_pid_t pid) * separate Ruby thread whose sole job is to reap the status of the * process _pid_ when it terminates. Use <code>detach</code> * only when you do not intent to explicitly wait for the child to - * terminate. <code>detach</code> only checks the status - * periodically (currently once each second). + * terminate. * * The waiting thread returns the exit status of the detached process * when it terminates, so you can use <code>Thread#join</code> to |