start of mojo conversion
[spider.git] / perl / Msg.pm
index 57880ed112f825f9d445a4bdf7397375f4ac0835..d62cb744034377f54592a3a7233571f9e88f842d 100644 (file)
@@ -5,69 +5,27 @@
 #
 # I have modified it to suit my devious purposes (Dirk Koopman G1TLH)
 #
-# $Id$
+#
 #
 
 package Msg;
 
 use strict;
 
-use vars qw($VERSION $BRANCH);
-$VERSION = sprintf( "%d.%03d", q$Revision$ =~ /(\d+)\.(\d+)/ );
-$BRANCH = sprintf( "%d.%03d", q$Revision$ =~ /\d+\.\d+\.(\d+)\.(\d+)/ ) || 0;
-$main::build += $VERSION;
-$main::branch += $BRANCH;
+use DXUtil;
+
+use Mojo::IOLoop;
+use Mojo::IOLoop::Stream;
 
-use IO::Select;
-use IO::Socket;
 use DXDebug;
 use Timer;
 
-use vars qw(%rd_callbacks %wt_callbacks %er_callbacks $rd_handles $wt_handles $er_handles $now %conns $noconns $blocking_supported $cnum);
+use vars qw($now %conns $noconns $cnum $total_in $total_out);
 
-%rd_callbacks = ();
-%wt_callbacks = ();
-%er_callbacks = ();
-$rd_handles   = IO::Select->new();
-$wt_handles   = IO::Select->new();
-$er_handles   = IO::Select->new();
+$total_in = $total_out = 0;
 
 $now = time;
 
-BEGIN {
-    # Checks if blocking is supported
-    eval {
-        require POSIX; POSIX->import(qw(O_NONBLOCK F_SETFL F_GETFL))
-    };
-       if ($@ || $main::is_win) {
-#              print STDERR "POSIX Blocking *** NOT *** supported $@\n";
-               $blocking_supported = 0;
-       } else {
-               $blocking_supported = 1;
-#              print STDERR "POSIX Blocking enabled\n";
-       }
-
-
-       # import as many of these errno values as are available
-       eval {
-               require Errno; Errno->import(qw(EAGAIN EINPROGRESS EWOULDBLOCK));
-       };
-       # http://support.microsoft.com/support/kb/articles/Q150/5/37.asp
-       # defines EINPROGRESS as 10035.  We provide it here because some
-       # Win32 users report POSIX::EINPROGRESS is not vendor-supported.
-       if ($^O eq 'MSWin32') { 
-               eval '*EINPROGRESS = sub { 10036 };';
-               eval '*EWOULDBLOCK = *EAGAIN = sub { 10035 };';
-               $blocking_supported = 1;
-       } 
-}
-
-my $w = $^W;
-$^W = 0;
-my $eagain = eval {EAGAIN()};
-my $einprogress = eval {EINPROGRESS()};
-my $ewouldblock = eval {EWOULDBLOCK()};
-$^W = $w;
 $cnum = 0;
 
 
@@ -95,7 +53,7 @@ sub new
 
        $noconns++;
        
-       dbg("Connection created ($noconns)") if isdbg('connll');
+       dbg("$class Connection created (total $noconns)") if isdbg('connll');
        return bless $conn, $class;
 }
 
@@ -103,39 +61,21 @@ sub set_error
 {
        my $conn = shift;
        my $callback = shift;
-       $conn->{eproc} = $callback;
-       set_event_handler($conn->{sock}, error => $callback) if exists $conn->{sock};
+       $conn->{sock}->on(error => sub {my ($stream, $err) = @_; $callback->($conn, $err);});
 }
 
-sub set_rproc
+sub set_on_eof
 {
        my $conn = shift;
        my $callback = shift;
-       $conn->{rproc} = $callback;
+       $conn->{sock}->on(close => sub {$callback->($conn);});
 }
 
-sub blocking
+sub set_rproc
 {
-       return unless $blocking_supported;
-
-       # Make the handle stop blocking, the Windows way.
-       if ($^O eq 'MSWin32') { 
-        my $set_it = $_[1];
-               
-        # 126 is FIONBIO (some docs say 0x7F << 16)
-        ioctl( $_[0],
-               0x80000000 | (4 << 16) | (ord('f') << 8) | 126,
-               $set_it
-             ) or confess "Can't set the handle non-blocking: $!";
-       }
-       
-       my $flags = fcntl ($_[0], F_GETFL, 0);
-       if ($_[1]) {
-               $flags &= ~O_NONBLOCK;
-       } else {
-               $flags |= O_NONBLOCK;
-       }
-       fcntl ($_[0], F_SETFL, $flags);
+       my $conn = shift;
+       my $callback = shift;
+       $conn->{rproc} = $callback;
 }
 
 # save it
@@ -148,11 +88,11 @@ sub conns
        if (ref $pkg) {
                $call = $pkg->{call} unless $call;
                return undef unless $call;
-               dbg("changing $pkg->{call} to $call") if isdbg('connll') && exists $pkg->{call} && $call ne $pkg->{call};
+               dbg((ref $pkg) . " changing $pkg->{call} to $call") if isdbg('connll') && exists $pkg->{call} && $call ne $pkg->{call};
                delete $conns{$pkg->{call}} if exists $pkg->{call} && exists $conns{$pkg->{call}} && $pkg->{call} ne $call; 
                $pkg->{call} = $call;
                $ref = $conns{$call} = $pkg;
-               dbg("Connection $pkg->{cnum} $call stored") if isdbg('connll');
+               dbg((ref $pkg) . " Connection $pkg->{cnum} $call stored") if isdbg('connll');
        } else {
                $ref = $conns{$call};
        }
@@ -171,6 +111,21 @@ sub pid_gone
        }
 }
 
+sub ax25
+{
+       my $conn = shift;
+       return $conn->{csort} eq 'ax25';
+}
+
+sub peerhost
+{
+       my $conn = shift;
+       $conn->{peerhost} ||= 'ax25' if $conn->ax25;
+       $conn->{peerhost} ||= $conn->{sock}->handle->peerhost if $conn->{sock};
+       $conn->{peerhost} ||= 'UNKNOWN';
+       return $conn->{peerhost};
+}
+
 #-----------------------------------------------------------------
 # Send side routines
 sub connect {
@@ -184,32 +139,76 @@ sub connect {
        $conn->{peerhost} = $to_host;
        $conn->{peerport} = $to_port;
        $conn->{sort} = 'Outgoing';
+
+       dbg((ref $conn) . " connecting $conn->{cnum} to $to_host:$to_port") if isdbg('connll');
        
-    # Create a new internet socket
-    my $sock = IO::Socket::INET->new();
-    return undef unless $sock;
+       my $sock;
+       $conn->{sock} = $sock = Mojo::IOLoop::Client->new;
+       $sock->on(connect => sub { dbg((ref $conn) . " connected $conn->{cnum} to $to_host:$to_port") if isdbg('connll');}, 
+                         error => {$conn->disconnect},
+                         close => {$conn->disconnect});
        
-       my $proto = getprotobyname('tcp');
-       $sock->socket(AF_INET, SOCK_STREAM, $proto) or return undef;
+       $sock->connect(address => $to_host, port => $to_port);
        
-       blocking($sock, 0);
-       $conn->{blocking} = 0;
+       dbg((ref $conn) . " connected $conn->{cnum} to $to_host:$to_port") if isdbg('connll');
 
-       my $ip = gethostbyname($to_host);
-#      my $r = $sock->connect($to_port, $ip);
-       my $r = connect($sock, pack_sockaddr_in($to_port, $ip));
-       return undef unless $r || _err_will_block($!);
-       
-       $conn->{sock} = $sock;
-    
     if ($conn->{rproc}) {
-        my $callback = sub {$conn->_rcv};
-        set_event_handler ($sock, read => $callback);
+               $sock->on(read => sub {my ($stream, $msg) = @_; $conn->_rcv($msg);} );
     }
     return $conn;
 }
 
-sub disconnect {
+sub start_program
+{
+       my ($conn, $line, $sort) = @_;
+       my $pid;
+       
+#      local $^F = 10000;              # make sure it ain't closed on exec
+#      my ($a, $b) = $io_socket->socketpair(AF_UNIX, SOCK_STREAM, PF_UNSPEC);
+#      if ($a && $b) {
+#              $a->autoflush(1);
+#              $b->autoflush(1);
+#              $pid = fork;
+#              if (defined $pid) {
+#                      if ($pid) {
+#                              close $b;
+#                              $conn->{sock} = $a;
+#                              $conn->{csort} = $sort;
+#                              $conn->{lineend} = "\cM" if $sort eq 'ax25';
+#                              $conn->{pid} = $pid;
+#                              if ($conn->{rproc}) {
+#                                      my $callback = sub {$conn->_rcv};
+#                                      Msg::set_event_handler ($a, read => $callback);
+#                              }
+#                              dbg("connect $conn->{cnum}: started pid: $conn->{pid} as $line") if isdbg('connect');
+#                      } else {
+#                              $^W = 0;
+#                              dbgclose();
+#                              STDIN->close;
+#                              STDOUT->close;
+#                              STDOUT->close;
+#                              *STDIN = IO::File->new_from_fd($b, 'r') or die;
+#                              *STDOUT = IO::File->new_from_fd($b, 'w') or die;
+#                              *STDERR = IO::File->new_from_fd($b, 'w') or die;
+#                              close $a;
+#                              unless ($main::is_win) {
+#                                      #                                               $SIG{HUP} = 'IGNORE';
+#                                      $SIG{HUP} = $SIG{CHLD} = $SIG{TERM} = $SIG{INT} = 'DEFAULT';
+#                                      alarm(0);
+#                              }
+#                              exec "$line" or dbg("exec '$line' failed $!");
+#                      } 
+#              } else {
+#                      dbg("cannot fork for $line");
+#              }
+#      } else {
+#              dbg("no socket pair $! for $line");
+#      }
+       return $pid;
+}
+
+sub disconnect 
+{
     my $conn = shift;
        return if exists $conn->{disconnecting};
 
@@ -225,7 +224,7 @@ sub disconnect {
                delete $conns{$call} if $ref && $ref == $conn;
        }
        $call ||= 'unallocated';
-       dbg("Connection $conn->{cnum} $call disconnected") if isdbg('connll');
+       dbg((ref $conn) . " Connection $conn->{cnum} $call disconnected") if isdbg('connll');
        
        # get rid of any references
        for (keys %$conn) {
@@ -235,188 +234,100 @@ sub disconnect {
        }
 
        if (defined($sock)) {
-               set_event_handler ($sock, read => undef, write => undef, error => undef);
-               shutdown($sock, 3);
-               close($sock);
+               $sock->remove;
        }
        
        unless ($main::is_win) {
                kill 'TERM', $conn->{pid} if exists $conn->{pid};
        }
+}
 
+sub _send_stuff
+{
+       my $conn = shift;
+       my $rq = $conn->{outqueue};
+    my $sock = $conn->{sock};
+       while (@$rq) {
+               my $data = shift @$rq;
+               my $lth = length $data;
+               my $call = $conn->{call} || 'none';
+               if (isdbg('raw')) {
+                       if (isdbg('raw')) {
+                               dbgdump('raw', "$call send $lth: ", $lth);
+                       }
+               }
+               if (defined $sock && !$sock->destroyed) {
+                       $sock->write($data);
+                       $total_out = $lth;
+               } else {
+                       dbg("_send_stuff $call ending data ignored: $data");
+               }
+       }
 }
 
 sub send_now {
     my ($conn, $msg) = @_;
     $conn->enqueue($msg);
-    $conn->_send (1); # 1 ==> flush
+    _send_stuff($conn);
 }
 
 sub send_later {
-    my ($conn, $msg) = @_;
-    $conn->enqueue($msg);
-    my $sock = $conn->{sock};
-    return unless defined($sock);
-    set_event_handler ($sock, write => sub {$conn->_send(0)});
+       goto &send_now;
 }
 
 sub enqueue {
     my $conn = shift;
-    push (@{$conn->{outqueue}}, defined $_[0] ? $_[0] : '');
-}
-
-sub _send {
-    my ($conn, $flush) = @_;
-    my $sock = $conn->{sock};
-    return unless defined($sock);
-    my $rq = $conn->{outqueue};
-
-    # If $flush is set, set the socket to blocking, and send all
-    # messages in the queue - return only if there's an error
-    # If $flush is 0 (deferred mode) make the socket non-blocking, and
-    # return to the event loop only after every message, or if it
-    # is likely to block in the middle of a message.
-
-       if ($conn->{blocking} != $flush) {
-               blocking($sock, $flush);
-               $conn->{blocking} = $flush;
-       }
-    my $offset = (exists $conn->{send_offset}) ? $conn->{send_offset} : 0;
-
-    while (@$rq) {
-        my $msg            = $rq->[0];
-               my $mlth           = length($msg);
-        my $bytes_to_write = $mlth - $offset;
-        my $bytes_written  = 0;
-               confess("Negative Length! msg: '$msg' lth: $mlth offset: $offset") if $bytes_to_write < 0;
-        while ($bytes_to_write > 0) {
-            $bytes_written = syswrite ($sock, $msg,
-                                       $bytes_to_write, $offset);
-            if (!defined($bytes_written)) {
-                if (_err_will_block($!)) {
-                    # Should happen only in deferred mode. Record how
-                    # much we have already sent.
-                    $conn->{send_offset} = $offset;
-                    # Event handler should already be set, so we will
-                    # be called back eventually, and will resume sending
-                    return 1;
-                } else {    # Uh, oh
-                                       &{$conn->{eproc}}($conn, $!) if exists $conn->{eproc};
-                                       $conn->disconnect;
-                    return 0; # fail. Message remains in queue ..
-                }
-            } elsif (isdbg('raw')) {
-                               my $call = $conn->{call} || 'none';
-                               dbgdump('raw', "$call send $bytes_written: ", $msg);
-                       }
-            $offset         += $bytes_written;
-            $bytes_to_write -= $bytes_written;
-        }
-        delete $conn->{send_offset};
-        $offset = 0;
-        shift @$rq;
-        #last unless $flush; # Go back to select and wait
-                            # for it to fire again.
-    }
-    # Call me back if queue has not been drained.
-    unless (@$rq) {
-        set_event_handler ($sock, write => undef);
-               if (exists $conn->{close_on_empty}) {
-                       &{$conn->{eproc}}($conn, undef) if exists $conn->{eproc};
-                       $conn->disconnect; 
-               }
-    }
-    1;  # Success
+    push @{$conn->{outqueue}}, defined $_[0] ? $_[0] : '';
 }
 
-sub dup_sock
+sub _err_will_block 
 {
-       my $conn = shift;
-       my $oldsock = $conn->{sock};
-       my $rc = $rd_callbacks{$oldsock};
-       my $wc = $wt_callbacks{$oldsock};
-       my $ec = $er_callbacks{$oldsock};
-       my $sock = $oldsock->new_from_fd($oldsock, "w+");
-       if ($sock) {
-               set_event_handler($oldsock, read=>undef, write=>undef, error=>undef);
-               $conn->{sock} = $sock;
-               set_event_handler($sock, read=>$rc, write=>$wc, error=>$ec);
-               $oldsock->close;
-       }
-}
-
-sub _err_will_block {
-       return 0 unless $blocking_supported;
-       return ($_[0] == $eagain || $_[0] == $ewouldblock || $_[0] == $einprogress);
+       return 0;
 }
 
 sub close_on_empty
 {
        my $conn = shift;
-       $conn->{close_on_empty} = 1;
+       $conn->{sock}->on(drain => sub {$conn->disconnect;});
 }
 
 #-----------------------------------------------------------------
 # Receive side routines
 
-sub new_server {
-    @_ == 4 || die "Msg->new_server (myhost, myport, login_proc\n";
-    my ($pkg, $my_host, $my_port, $login_proc) = @_;
-       my $self = $pkg->new($login_proc);
+sub new_server 
+{
+#    @_ == 4 || die "Msg->new_server (myhost, myport, login_proc)\n";
+       my ($pkg, $my_host, $my_port, $login_proc) = @_;
+       my $conn = $pkg->new($login_proc);
+       
+    $conn->{sock} = Mojo::IOLoop::Server->new;
+       $conn->{sock}->on(accept=>sub{$conn->new_client()});
+       $conn->{sock}->listen(address=>$my_host, port=>$my_port);
        
-    $self->{sock} = IO::Socket::INET->new (
-                                          LocalAddr => "$my_host:$my_port",
-#                                          LocalPort => $my_port,
-                                          Listen    => SOMAXCONN,
-                                          Proto     => 'tcp',
-                                          Reuse => 1);
-    die "Could not create socket: $! \n" unless $self->{sock};
-    set_event_handler ($self->{sock}, read => sub { $self->new_client }  );
-       return $self;
+    die "Could not create socket: $! \n" unless $conn->{sock};
+       return $conn;
 }
 
-eval "use Socket qw(IPPROTO_TCP TCP_NODELAY)";
-if ($@) {
-       sub IPPROTO_TCP {6;}
-       sub TCP_NODELAY {1;};
-}
 
 sub nolinger
 {
        my $conn = shift;
-
-       if (isdbg('sock')) {
-               my ($l, $t) = unpack "ll", getsockopt($conn->{sock}, SOL_SOCKET, SO_LINGER); 
-               my $k = unpack 'l', getsockopt($conn->{sock}, SOL_SOCKET, SO_KEEPALIVE);
-               my $n = unpack "l", getsockopt($conn->{sock}, IPPROTO_TCP, TCP_NODELAY);
-               dbg("Linger is: $l $t, keepalive: $k, nagle: $n");
-       }
-
-       setsockopt($conn->{sock}, SOL_SOCKET, SO_LINGER, pack("ll", 0, 0)) or confess "setsockopt linger: $!";
-       setsockopt($conn->{sock}, SOL_SOCKET, SO_KEEPALIVE, 1) or confess "setsockopt keepalive: $!";
-       setsockopt($conn->{sock}, IPPROTO_TCP, TCP_NODELAY, 1) or confess "setsockopt: $!" unless $main::iswin;
-       $conn->{sock}->autoflush(0);
-       
-       if (isdbg('sock')) {
-               my ($l, $t) = unpack "ll", getsockopt($conn->{sock}, SOL_SOCKET, SO_LINGER); 
-               my $k = unpack 'l', getsockopt($conn->{sock}, SOL_SOCKET, SO_KEEPALIVE);
-               my $n = unpack "l", getsockopt($conn->{sock}, IPPROTO_TCP, TCP_NODELAY);
-               dbg("Linger is: $l $t, keepalive: $k, nagle: $n");
-       }
 }
 
 sub dequeue
 {
        my $conn = shift;
-
-       if ($conn->{msg} =~ /\n/) {
-               my @lines = split /\r?\n/, $conn->{msg};
-               if ($conn->{msg} =~ /\n$/) {
+       return if $conn->{disconnecting};
+       
+       if ($conn->{msg} =~ /\cJ/) {
+               my @lines = split /\cM?\cJ/, $conn->{msg};
+               if ($conn->{msg} =~ /\cM?\cJ$/) {
                        delete $conn->{msg};
                } else {
                        $conn->{msg} = pop @lines;
                }
                for (@lines) {
+                       last if $conn->{disconnecting};
                        &{$conn->{rproc}}($conn, defined $_ ? $_ : '');
                }
        }
@@ -430,13 +341,14 @@ sub _rcv {                     # Complement to _send
     return unless defined($sock);
 
        my @lines;
-       if ($conn->{blocking}) {
-               blocking($sock, 0);
-               $conn->{blocking} = 0;
-       }
+#      if ($conn->{blocking}) {
+#              blocking($sock, 0);
+#              $conn->{blocking} = 0;
+#      }
        $bytes_read = sysread ($sock, $msg, 1024, 0);
        if (defined ($bytes_read)) {
                if ($bytes_read > 0) {
+                       $total_in += $bytes_read;
                        if (isdbg('raw')) {
                                my $call = $conn->{call} || 'none';
                                dbgdump('raw', "$call read $bytes_read: ", $msg);
@@ -482,37 +394,30 @@ FINISH:
 
 sub new_client {
        my $server_conn = shift;
-    my $sock = $server_conn->{sock}->accept();
-       if ($sock) {
-               my $conn = $server_conn->new($server_conn->{rproc});
-               $conn->{sock} = $sock;
-               blocking($sock, 0);
-               $conn->nolinger;
-               $conn->{blocking} = 0;
-               my ($rproc, $eproc) = &{$server_conn->{rproc}} ($conn, $conn->{peerhost} = $sock->peerhost(), $conn->{peerport} = $sock->peerport());
-               $conn->{sort} = 'Incoming';
-               if ($eproc) {
-                       $conn->{eproc} = $eproc;
-                       set_event_handler ($sock, error => $eproc);
-               }
-               if ($rproc) {
-                       $conn->{rproc} = $rproc;
-                       my $callback = sub {$conn->_rcv};
-                       set_event_handler ($sock, read => $callback);
-               } else {  # Login failed
-                       &{$conn->{eproc}}($conn, undef) if exists $conn->{eproc};
-                       $conn->disconnect();
-               }
-       } else {
-               dbg("Msg: error on accept ($!)") if isdbg('err');
+       my $client = shift;
+       
+       my $conn = $server_conn->new($server_conn->{rproc});
+       my $sock = $conn->{sock} = Mojo::IOLoop::Stream->new($client);
+       $sock->on(read => sub {$conn->_rcv($_[1])});
+       dbg((ref $conn) . "accept $conn->{cnum} from $conn->{peerhost} $conn->{peerport}") if isdbg('connll');
+
+       my ($rproc, $eproc) = &{$server_conn->{rproc}} ($conn, $conn->{peerhost} = $client->peerhost, $conn->{peerport} = $client->peerport);
+       $conn->{sort} = 'Incoming';
+       if ($eproc) {
+               $conn->{eproc} = $eproc;
+       }
+       if ($rproc) {
+               $conn->{rproc} = $rproc;
+       } else {  # Login failed
+               &{$conn->{eproc}}($conn, undef) if exists $conn->{eproc};
+               $conn->disconnect();
        }
 }
 
 sub close_server
 {
        my $conn = shift;
-       set_event_handler ($conn->{sock}, read => undef, write => undef, error => undef );
-       $conn->{sock}->close;
+       delete $conn->{sock};
 }
 
 # close all clients (this is for forking really)
@@ -526,76 +431,24 @@ sub close_all_clients
 sub disable_read
 {
        my $conn = shift;
-       set_event_handler ($conn->{sock}, read => undef);
-       return $_[0] ? $conn->{disable_read} = $_[0] : $_[0];
+       return defined $_[0] ? $conn->{disable_read} = $_[0] : $_[0];
 }
 
+
 #
 #----------------------------------------------------
 # Event loop routines used by both client and server
 
 sub set_event_handler {
-    shift unless ref($_[0]); # shift if first arg is package name
-    my ($handle, %args) = @_;
-    my $callback;
-    if (exists $args{'write'}) {
-        $callback = $args{'write'};
-        if ($callback) {
-            $wt_callbacks{$handle} = $callback;
-            $wt_handles->add($handle);
-        } else {
-            delete $wt_callbacks{$handle};
-            $wt_handles->remove($handle);
-        }
-    }
-    if (exists $args{'read'}) {
-        $callback = $args{'read'};
-        if ($callback) {
-            $rd_callbacks{$handle} = $callback;
-            $rd_handles->add($handle);
-        } else {
-            delete $rd_callbacks{$handle};
-            $rd_handles->remove($handle);
-       }
-    }
-    if (exists $args{'error'}) {
-        $callback = $args{'error'};
-        if ($callback) {
-            $er_callbacks{$handle} = $callback;
-            $er_handles->add($handle);
-        } else {
-            delete $er_callbacks{$handle};
-            $er_handles->remove($handle);
-       }
-    }
-}
-
-sub event_loop {
-    my ($pkg, $loop_count, $timeout) = @_; # event_loop(1) to process events once
-    my ($conn, $r, $w, $e, $rset, $wset, $eset);
-    while (1) {
-       # Quit the loop if no handles left to process
-        last unless ($rd_handles->count() || $wt_handles->count());
-        
-               ($rset, $wset, $eset) = IO::Select->select($rd_handles, $wt_handles, $er_handles, $timeout);
-               
-        foreach $e (@$eset) {
-            &{$er_callbacks{$e}}($e) if exists $er_callbacks{$e};
-        }
-        foreach $r (@$rset) {
-            &{$rd_callbacks{$r}}($r) if exists $rd_callbacks{$r};
-        }
-        foreach $w (@$wset) {
-            &{$wt_callbacks{$w}}($w) if exists $wt_callbacks{$w};
-        }
-
-               Timer::handler;
-               
-        if (defined($loop_count)) {
-            last unless --$loop_count;
-        }
-    }
+       my $sock = shift;
+       my %args = @_;
+       my ($pkg, $fn, $line) = caller;
+       my $s;
+       foreach (my ($k,$v) = each %args) {
+               $s .= "$k => $v, ";
+       }
+       $s =~ s/[\s,]$//;
+       dbg("Msg::set_event_handler called from ${pkg}::${fn} line $line doing $s");
 }
 
 sub sleep
@@ -603,7 +456,7 @@ sub sleep
        my ($pkg, $interval) = @_;
        my $now = time;
        while (time - $now < $interval) {
-               $pkg->event_loop(10, 0.01);
+               sleep 1;
        }
 }
 
@@ -613,8 +466,8 @@ sub DESTROY
        my $call = $conn->{call} || 'unallocated';
        my $host = $conn->{peerhost} || '';
        my $port = $conn->{peerport} || '';
-       dbg("Connection $conn->{cnum} $call [$host $port] being destroyed") if isdbg('connll');
        $noconns--;
+       dbg((ref $conn) . " Connection $conn->{cnum} $call [$host $port] being destroyed (total $noconns)") if isdbg('connll');
 }
 
 1;