This queue is for tickets about the Parallel-Iterator CPAN distribution.

Report information
The Basics
Id:
67597
Status:
new
Priority:
Low/Low

People
Owner:
Nobody in particular
Requestors:
rganski [...] gracenote.com
Cc:
AdminCc:

BugTracker
Severity:
(no value)
Broken in:
(no value)
Fixed in:
(no value)



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



This service runs on Request Tracker, is sponsored by The Perl Foundation, and maintained by Best Practical Solutions.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.