Skip Menu |

This queue is for tickets about the Net-Async-HTTP CPAN distribution.

Report information
The Basics
Id: 101460
Status: resolved
Priority: 0/
Queue: Net-Async-HTTP
People
Owner: Nobody in particular
Requestors: TEAM [...] cpan.org
Cc:
AdminCc:
Bug Information
Severity: Unimportant
Broken in: 0.37
Fixed in: 0.39

History Show all quoted text
  Sun Jan 11 12:29:27 2015 TEAM [...] cpan.org - Ticket created
Subject: Redundant ->on_cancel in NaHTTP::Connection
NaHTTP::Connection lines 248-249: $f->on_ready( $remove_timer ); $f->on_cancel( $remove_timer ); That ->on_cancel should be redundant, I think?
  Mon Jul 13 09:37:38 2015 leonerd-cpan [...] leonerd.org.uk - Correspondence added
On Sun Jan 11 12:29:27 2015, TEAM wrote: Show quoted text
> NaHTTP::Connection lines 248-249: > > $f->on_ready( $remove_timer ); > $f->on_cancel( $remove_timer ); > > That ->on_cancel should be redundant, I think?
Ah yes. The Future docs don't make it excessively clear, but yes - cancellation is a kind of readiness that on_ready will respond to. -- Paul Evans
  Mon Jul 13 09:37:39 2015 The RT System itself - Status changed from 'new' to 'open'
  Mon Jul 13 09:37:40 2015 leonerd-cpan [...] leonerd.org.uk - Status changed from 'open' to 'patched'
  Tue Jul 18 09:12:17 2017 leonerd-cpan [...] leonerd.org.uk - Correspondence added
Was released in 0.39 -- Paul Evans
  Tue Jul 18 09:12:23 2017 leonerd-cpan [...] leonerd.org.uk - Status changed from 'patched' to 'resolved'
  Tue Jul 18 09:12:28 2017 leonerd-cpan [...] leonerd.org.uk - Fixed in 0.39 added