https://issues.apache.org/bugzilla/show_bug.cgi?id=45687





--- Comment #4 from Jeff Trawick <[EMAIL PROTECTED]>  2008-08-29 12:35:29 PST 
---
In general you're WAAY ahead of me and I don't have much energy to spare on
this...  if somebody with time avail is watching this space, pick it up and run

Another note on your patch:

in xlate_brigade(), instead of adding a separate check, change the

b == APR_BRIGADE_SINTENEL(bb) || APR_BUCKET_IS_EOS(b)

to

b == APR_BRIGADE_SENTINEL(bb) || APR_BUCKET_IS_METADATA(b)

since EOS is a type of METADATA; we just weren't handling the general condition
here

I withdraw my prior comment about "searching" for metadata; I now see that it
is just picking off an arbitrary number of metadata buckets from the start;
it should function the same just picking off the first, in case that makes it
simpler to read

fwiw, xlate_brigade() should work whether or not the caller is
xlate_in_filter() (or did at one time)


-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to