Regression introduced in trunk r13201 (large rock merge)

2014-02-03 Thread Kinkie
Hi, it appears that the large rock merge has introduced a regression, probably related to the handling of keepalive and/or about finishing to send an object to a client. I have confirmed that the issue is NOT present in revno 13200, but it is present in r13203 (r13201 and r13202 do not

Re: Regression introduced in trunk r13201 (large rock merge)

2014-02-03 Thread Amos Jeffries
FWIW; this seems to be the same issue is under discussion in squid-users thread rock store: a bug or ...?. Cc'ing Henrik Lidström and Nikolai Gorchilov. Henrik, Nokolai: if you could followup to squid-dev in future about this please. On 2014-02-04 05:25, Kinkie wrote: Hi, it appears

Re: Regression introduced in trunk r13201 (large rock merge)

2014-02-03 Thread Alex Rousskov
On 02/03/2014 01:55 PM, Amos Jeffries wrote: FWIW; this seems to be the same issue is under discussion in squid-users thread rock store: a bug or ...?. Cc'ing Henrik Lidström and Nikolai Gorchilov. Henrik, Nokolai: if you could followup to squid-dev in future about this please. On

Re: [PATCH] refactor Vector

2014-02-03 Thread Alex Rousskov
On 02/02/2014 02:42 PM, Amos Jeffries wrote: in src/HttpHdrRange.cc * This pattern happens several times throughout this patch: -while (!specs.empty()) -delete specs.pop_back(); +while (!specs.empty()) { +delete specs.back(); +