Re: [PATCH] Fix an assert with side effects in ServerStateData::handleAdaptedHeader

From: Amos Jeffries <squid3_at_treenet.co.nz>
Date: Thu, 25 Jul 2013 19:22:41 +1200

On 25/07/2013 6:51 p.m., Nathan Hoad wrote:
> This patch simply moves the
> adaptedBodySource->setConsumerIfNotLate(this) call out of an assert()
> in ServerStateData::handleAdaptedHeader.
>
> setConsumerIfNotLate conditionally schedules some async calls, so the
> call definitely shouldn't be removed by the precompiler.

Indeed. Applied, thank you

Amos
Received on Thu Jul 25 2013 - 07:22:50 MDT

This message: [ Message body ]
Next message: Henrik Nordström: "Re: What is The logic of Vary Headers cachiness?"
Previous message: Amos Jeffries: "Re: What is The logic of Vary Headers cachiness?"
In reply to: Nathan Hoad: "[PATCH] Fix an assert with side effects in ServerStateData::handleAdaptedHeader"

Contemporary messages sorted: [ by date ] [ by thread ] [ by subject ] [ by author ] [ by messages with attachments ]

This archive was generated by hypermail 2.2.0 : Thu Jul 25 2013 - 12:00:11 MDT