Cliff Woolley wrote:

>On Thu, 28 Feb 2002, Cliff Woolley wrote:
>
>>On Thu, 28 Feb 2002, Greg Stein wrote:
>>
>>>>/* prepend ctx->ssi_tag_brigade onto bb */
>>>>APR_BRIGADE_CONCAT(ctx->ssi_tag_brigade, bb);
>>>>APR_BRIGADE_CONCAT(bb, ctx->ssi_tag_brigade);
>>>>
>>>That is *very* obtuse. I can't imagine anybody figuring out that idiom
>>>means "prepend". Yes, please add an appropriate macro.
>>>
>>Okay, will add.
>>
>
>Committed.  Brian, can you please run your tests on it for me?  I've yet
>to find the exact test case to get into that block of code.
>

I just finished testing against my test cases that exercise this
part of the code, and the new version using APR_BRIGADE_PREPEND
works fine.

All my test cases for this are variants of a technique that Ian
created: use mod_bucketeer in front of mod_include, and insert
ctrl-B's (force new bucket), ctrl-P's (pass brigade now), and
ctrl-F's (flush brigade) after '<' symbols that aren't the start
of a "<!--#".

This exercises the code in mod_include that keeps track of
<!--# directives that span brigades.

--Brian


Reply via email to