Subject: | Documentation error |
Date: | Tue, 19 Apr 2011 16:29:26 -0700 |
To: | "bug-parallel-iterator [...] rt.cpan.org" <bug-parallel-iterator [...] rt.cpan.org> |
From: | Richard Ganski <rganski [...] gracenote.com> |
Hi, there is an error in the documentation at http://search.cpan.org/~andya/Parallel-Iterator-1.00/lib/Parallel/Iterator.pm:
The subroutine fetch is defined as
sub fetch {
my $url = shift;
my $resp = $ua->get($url);
return unless $resp->is_success;
return $resp->content;
};
when discussing building one's own iterator. Then it is called unchanged when passed to iterate and iterate_as_array. But when used in conjunction with those routines, fetch should be modified to get $url from the second argument:
sub fetch {
my $id = shift;
my $url = shift; # or my ($id, $url) = @_;
my $resp = $ua->get($url);
return unless $resp->is_success;
return $resp->content;
};
It's a great module!
Thanks
Rich Ganski
Message body is not shown because it is too large.