Skip Menu |
 

This queue is for tickets about the POE-Component-Client-HTTP CPAN distribution.

Report information
The Basics
Id: 5810
Status: resolved
Priority: 0/
Queue: POE-Component-Client-HTTP

People
Owner: Nobody in particular
Requestors: sbeck [...] gossamer-threads.com
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)



Subject: Streaming size
From: Scott Beck <sbeck [...] gossamer-threads.com>
To: bug-POE-Component-Client-HTTP [...] rt.cpan.org
Date: Thu, 25 Mar 2004 15:37:52 -0800
Download (untitled) / with headers
text/plain 726b
Hello, I've been going through the POE::Component::Client::HTTP code considering using it for POE::Component::Client::DAAP which I am currently working on. I will need the Streaming part of it to work properly. I noticed that you set the streaming size in spawn() in the heap as $heap->{stream}. But in the place you create the Wheel::ReadWrite object you use $heap->{streaming}. Around line 577 you have: my $block_size = $heap->{streaming} || DEFAULT_BLOCK_SIZE; $block_size = DEFAULT_BLOCK_SIZE if $block_size < 1; This and in several place you use $heap->{streaming} instead of $heap->{stream}. Am I missing something or is that a bug? Cheers, Scott -- Scott Beck <sbeck@gossamer-threads.com> Gossamer Threads
Download (untitled) / with headers
text/plain 272b
I believe the "streams" vs. "streaming" to be a bug. I've replaced streams with streaming throughout HTTP.pm, and the tests continue to work, so that's the way it will be in the next release. Thanks for reporting this; sorry it's been pending so long. -- Rocco Caputo


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

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