On Mon, Jun 4, 2018 at 11:23 AM, Luca Toscano <toscano.luca@xxxxxxxxx> wrote:
> To keep archives happy: opened
bugzilla/show_bug.cgi?id=62362and 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?