X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?p=scpubgit%2FObject-Remote.git;a=blobdiff_plain;f=lib%2FObject%2FRemote%2FReadChannel.pm;h=ed29f582628139db13a2b37a0abfc902c960c15d;hp=6b273482755bc38a06c34feed7829b6dac5d70ca;hb=8c3529062a426181861d58ee59fb8f10e0be68e5;hpb=7790ca36241acfa9379cb2b89b68c9f4b0da2371 diff --git a/lib/Object/Remote/ReadChannel.pm b/lib/Object/Remote/ReadChannel.pm index 6b27348..ed29f58 100644 --- a/lib/Object/Remote/ReadChannel.pm +++ b/lib/Object/Remote/ReadChannel.pm @@ -1,11 +1,12 @@ package Object::Remote::ReadChannel; use CPS::Future; -use Scalar::Util qw(weaken); -use Object::Remote::Logging qw(:log :dlog); -use POSIX; +use Scalar::Util qw(weaken openhandle); +use Object::Remote::Logging qw(:log :dlog router ); use Moo; +BEGIN { router()->exclude_forwarding } + has fh => ( is => 'ro', required => 1, trigger => sub { @@ -33,22 +34,20 @@ sub _receive_data_from { Dlog_trace { "Preparing to read data from $_" } $fh; my $rb = $self->_receive_data_buffer; my $len = sysread($fh, $$rb, 32768, length($$rb)); - my $err = defined($len) ? '' : ": $!"; + my $err = defined($len) ? 'eof' : ": $!"; if (defined($len) and $len > 0) { log_trace { "Read $len bytes of data" }; while (my $cb = $self->on_line_call and $$rb =~ s/^(.*)\n//) { $cb->(my $line = $1); } - #TODO this isn't compatible with Windows but would be if - #EAGAIN was set to something that could never match - #if on Windows - } elsif ($! != EAGAIN) { + } else { log_trace { "Got EOF or error, this read channel is done" }; Object::Remote->current_loop ->unwatch_io( handle => $self->fh, on_read_ready => 1 ); + log_trace { "Invoking on_close_call() for dead read channel" }; $self->on_close_call->($err); } } @@ -57,11 +56,14 @@ sub DEMOLISH { my ($self, $gd) = @_; return if $gd; log_trace { "read channel is being demolished" }; + Object::Remote->current_loop ->unwatch_io( handle => $self->fh, on_read_ready => 1 ); + + } 1;