cURL / Mailing Lists / curl-library / Single Mail

curl-library

RE: Bug 2351645

From: Igor Novoseltsev <IgorN_at_radvision.com>
Date: Tue, 9 Dec 2008 08:15:26 +0200

> Or what about making Curl_getoff_all_pipelines() deal with the
situation
> properly and only set *_inuse to FALSE if the handle actually was head
- and
> thus work for either situation? See the attachment to this mail for my
> suggested fix. (diff against current CVS HEAD)
>
> Do you think this works?

If the handle is head, it is removed from pipelines without any fix at
all (from within singlesocket()).
Therefore in this case there is no need to call
Curl_getoff_all_pipelines() at all.

Despite that I would insert the latest (fixed) version of
Curl_getoff_all_pipelines,
Since it makes the code more readable.

Thanks,
Igor
Received on 2008-12-09