[squid-dev] [PATCH] HTTP Response Parser upgrade

Kinkie gkinkie at gmail.com
Mon Feb 23 10:03:03 UTC 2015


I've been testing it for a few days; +1.

On Mon, Feb 23, 2015 at 5:37 AM, Amos Jeffries <squid3 at treenet.co.nz> wrote:
> On 24/01/2015 8:37 a.m., Amos Jeffries wrote:
>> Updated patch including all changes made after previous review.
>>
>> Most significantly:
>>
>> * rearranging the cascade of Tokenizer calls into if-conditional
>> operations with token+delimiter pairs outlining the success case
>> explicitly then assuming failure. Earlier patch did the opposite;
>> determining significant failures and assuming success otherwise.
>>  - giving shorter code, and somewhat easier readability.
>>
>> * updating the Tokenizer int64 API to handle limited-length scans and
>> optional +/- symbols.
>>
>> Polygraph and Coadvisor testing shows no significant change since
>> previous patch.
>>
>> Amos
>>
>
> If there are no objections I would like to merge this in approx 3 days.
>
> Amos
>
> _______________________________________________
> squid-dev mailing list
> squid-dev at lists.squid-cache.org
> http://lists.squid-cache.org/listinfo/squid-dev



-- 
    Francesco


More information about the squid-dev mailing list