Hi Luca, On Mon, Jun 4, 2018 at 11:23 AM, Luca Toscano <toscano.l...@gmail.com> wrote: > > To keep archives happy: opened > https://bz.apache.org/bugzilla/show_bug.cgi?id=62362 and added a patch in > there, if anybody wants to review it and give me suggestions I'd be happy :)
The semantic of tmpbb is not very clear in your patch, it's both the brigade where buckets are saved (for the next call?) and the one that gets passed to the next filter finally. It doesn't look right to me, if tmpbb is to be forwarded in the same pass there is no need to change its buckets' lifetime. Couldn't we ap_save_brigade(f, &ctx->holdingbb, &ctx->tmpbb, ..) at the end of the filter only? Thanks, Yann.
