Re: [MERGE] c++-refactor HttpHdrCc

From: Kinkie <gkinkie_at_gmail.com>
Date: Tue, 4 Oct 2011 23:08:07 +0200

On Tue, Oct 4, 2011 at 11:05 PM, Amos Jeffries <squid3_at_treenet.co.nz> wrote:
> On Tue, 4 Oct 2011 14:00:14 +0200, Kinkie wrote:
>>
>> If you guys are OK, I'll merge the current implementation, then we can
>> alter it post-merge.
>>
>> Right now what Squid does is A1->B2 for all directives except for
>> max-stale, for which it does B4 (forward valueless). This  is probably
>> the worst thing that can be done.
>>
>> IMHVO the best thing we could do is forward things we can't parse
>> as-is (A2-B3). If we change nothing we can probably also keep a copy
>> of the raw directives, saves some cycles to reassemble it at pack-time
>> .
>
>
> I'm okay with an intermediary polish-only merge as long as its not adding or
> changing the behaviour mess.
>
> If it is changing behaviour then may as well make sure there are no known
> bugs in the post-merge result.

No behaviour changes.
These will have to wait until a consensus is reached.

Ok, merging.
Thanks

-- 
    /kinkie
Received on Tue Oct 04 2011 - 21:08:14 MDT

This archive was generated by hypermail 2.2.0 : Wed Oct 05 2011 - 12:00:03 MDT