Re: [RFC PATCH] Re: PAC serving

2010-07-16 Thread Alex Rousskov
On 07/15/2010 08:58 PM, Amos Jeffries wrote: > Alex Rousskov wrote: >> On 07/14/2010 08:02 AM, Amos Jeffries wrote: >>> Since no description beats the real code here is the current patch I >>> have. >>> >>> If you guys can't find any other major issues with this I'll go ahead >>> and make it build

Re: [RFC PATCH] Re: PAC serving

2010-07-15 Thread Amos Jeffries
Alex Rousskov wrote: On 07/14/2010 08:02 AM, Amos Jeffries wrote: Since no description beats the real code here is the current patch I have. If you guys can't find any other major issues with this I'll go ahead and make it build and test it out. I have some comments, but they should not stop

Re: [RFC PATCH] Re: PAC serving

2010-07-14 Thread Amos Jeffries
On Wed, 14 Jul 2010 15:34:18 -0600, Alex Rousskov wrote: > On 07/14/2010 08:02 AM, Amos Jeffries wrote: >> Since no description beats the real code here is the current patch I >> have. >> >> If you guys can't find any other major issues with this I'll go ahead >> and make it build and test it out

Re: PAC serving

2010-07-14 Thread Alex Rousskov
On 07/13/2010 07:02 PM, Henrik Nordström wrote: > ons 2010-07-14 klockan 00:53 + skrev Amos Jeffries: > >> Thats the plan. Those three objects I said we have to work with are the >> parameters available to internalStart(). > > My copy only have two parameters.. > > internalStart(HttpRequest

Re: [RFC PATCH] Re: PAC serving

2010-07-14 Thread Alex Rousskov
On 07/14/2010 08:02 AM, Amos Jeffries wrote: > Since no description beats the real code here is the current patch I have. > > If you guys can't find any other major issues with this I'll go ahead > and make it build and test it out. I have some comments, but they should not stop you from running

[RFC PATCH] Re: PAC serving

2010-07-14 Thread Amos Jeffries
Since no description beats the real code here is the current patch I have. If you guys can't find any other major issues with this I'll go ahead and make it build and test it out. Amos -- Please be using Current Stable Squid 2.7.STABLE9 or 3.1.5 === added file 'src/InternalServer.cc' --- src

Re: PAC serving

2010-07-13 Thread Henrik Nordström
ons 2010-07-14 klockan 00:53 + skrev Amos Jeffries: > Thats the plan. Those three objects I said we have to work with are the > parameters available to internalStart(). My copy only have two parameters.. internalStart(HttpRequest * request, StoreEntry * entry) > So far I have the else case