--- AnyEvent/lib/AnyEvent.pm 2007/11/02 19:12:02 1.30 +++ AnyEvent/lib/AnyEvent.pm 2007/11/09 19:34:32 1.33 @@ -176,8 +176,10 @@ =head2 CHILD PROCESS WATCHERS You can also listen for the status of a child process specified by the -C argument. The watcher will only trigger once. This works by -installing a signal handler for C. +C argument (or any child if the pid argument is 0). The watcher will +trigger as often as status change for the child are received. This works +by installing a signal handler for C. The callback will be called with +the pid and exit status (as returned by waitpid). Example: wait for pid 1333 @@ -197,7 +199,8 @@ The known classes so far are: - EV::AnyEvent based on EV (an interface to libev, best choice) + AnyEvent::Impl::CoroEV based on Coro::EV, best choice. + AnyEvent::Impl::EV based on EV (an interface to libev, also best choice). AnyEvent::Impl::Coro based on Coro::Event, second best choice. AnyEvent::Impl::Event based on Event, also second best choice :) AnyEvent::Impl::Glib based on Glib, second-best choice. @@ -262,8 +265,9 @@ our @REGISTRY; my @models = ( + [Coro::EV:: => AnyEvent::Impl::CoroEV::], + [EV:: => AnyEvent::Impl::EV::], [Coro::Event:: => AnyEvent::Impl::Coro::], - [EV:: => EV::AnyEvent::], [Event:: => AnyEvent::Impl::Event::], [Glib:: => AnyEvent::Impl::Glib::], [Tk:: => AnyEvent::Impl::Tk::], @@ -351,7 +355,7 @@ my $signal = uc $arg{signal} or Carp::croak "required option 'signal' is missing"; - $SIG_CB{$signal}{$arg{cb} += 0} = $arg{cb}; + $SIG_CB{$signal}{$arg{cb}} = $arg{cb}; $SIG{$signal} ||= sub { $_->() for values %{ $SIG_CB{$signal} || {} }; }; @@ -375,8 +379,9 @@ our $WNOHANG; sub _child_wait { - while (0 < (my $pid = waitpid -1, $WNOHANG)) { - $_->() for values %{ $PID_CB{$pid} || {} }, %{ $PID_CB{0} || {} }; + while (0 <= (my $pid = waitpid -1, $WNOHANG)) { + $_->($pid, $?) for (values %{ $PID_CB{$pid} || {} }), + (values %{ $PID_CB{0} || {} }); } undef $PID_IDLE; @@ -385,7 +390,7 @@ sub child { my (undef, %arg) = @_; - my $pid = uc $arg{pid} + defined (my $pid = $arg{pid} + 0) or Carp::croak "required option 'pid' is missing"; $PID_CB{$pid}{$arg{cb}} = $arg{cb};