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.
Amos
