Re: rflush() not working as documented?

2003-08-14 Thread Martin Wickman
On Wed, Aug 06, 2003 at 07:33:49AM -0700, Stas Bekman wrote: Martin Wickman wrote: [...] oops, sorry. Can you try with the latest cvs? Not at the moment. But I'll try in a few days. Your particular problem report was fine, what you have missed is the output of t/REPORT which tells us

Re: rflush() not working as documented?

2003-08-14 Thread Martin Wickman
Stas Bekman wrote: Geoffrey Young wrote: Martin Wickman wrote: Martin Wickman wrote: According to docs[1], $r-rflush() should create a new brigade with data. It does not. I've seen this also, but was never able to isolate a cause. rflush() works fine, it's possible that the issue with the

Re: rflush() not working as documented?

2003-08-14 Thread Martin Wickman
On Thu, Aug 07, 2003 at 04:46:51PM -0700, Stas Bekman wrote: Please try the latest mp2 cvs, I've added a new test t/api/rflush.t, it tests rflush explicitly (even though it's already used for exactly this purpose in several other tests). [...] does it work for you? I tried the latest CVS

Re: rflush() not working as documented?

2003-08-14 Thread Stas Bekman
Martin Wickman wrote: Stas Bekman wrote: Geoffrey Young wrote: Martin Wickman wrote: Martin Wickman wrote: According to docs[1], $r-rflush() should create a new brigade with data. It does not. I've seen this also, but was never able to isolate a cause. rflush() works fine, it's possible that

Re: rflush() not working as documented?

2003-08-14 Thread Stas Bekman
Martin Wickman wrote: I tried the latest CVS (modperl-2.0_20030810101543) and my code now works as expected, ie rflush() splits correctly into brigades. The t/api/rflush.t works as well btw. Excellent! Is it any different from your code? Your test-code is essentially the same as my code. I

Re: rflush() not working as documented?

2003-08-09 Thread Stas Bekman
Martin Wickman wrote: Please try the latest mp2 cvs, I've added a new test t/api/rflush.t, it tests rflush explicitly (even though it's already used for exactly this purpose in several other tests). It does exactly what your code does: sub response { my $r = shift; # just to make sure

Re: rflush() not working as documented?

2003-08-04 Thread Martin Wickman
Just checking that this did not get lost on the way. Anyone care to give me a hint? On Thu, Jul 31, 2003 at 10:17:06PM +0200, Martin Wickman wrote: Hello According to docs[1], $r-rflush() should create a new brigade with data. It does not. It seems the docs and/or my understanding of

Re: rflush() not working as documented?

2003-08-04 Thread Geoffrey Young
Martin Wickman wrote: Just checking that this did not get lost on the way. Anyone care to give me a hint? On Thu, Jul 31, 2003 at 10:17:06PM +0200, Martin Wickman wrote: Hello According to docs[1], $r-rflush() should create a new brigade with data. It does not. I've seen this also, but was