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
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
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