Hi Yann, 2018-06-07 12:59 GMT+02:00 Yann Ylavic <ylavic....@gmail.com>:
> On Thu, Jun 7, 2018 at 10:17 AM, Yann Ylavic <ylavic....@gmail.com> wrote: > > On Thu, Jun 7, 2018 at 9:21 AM, Yann Ylavic <ylavic....@gmail.com> > wrote: > >> > >> Feel free to take what can serve you, or burn it ;) > > > > Burn it! Here is v2, hopefully a less buggy :) > > FWIW, a v3 with less changes and a few comments where needed. > Less changes because it keeps using tmpbb in the RATE_LIMIT loop (with > APR_BRIGADE_CONCAT(ctx->tmpbb, bb) first). > HTH... > All my consistency tests are green, and after a first (and quick!) pass of the code in your patch I can definitely say that your version is much better and coincise, I like it! Will try to review it in detail during the next days and then I'll commit it. Thanks! Luca