cURL / Mailing Lists / curl-library / Single Mail

curl-library

Re: [PATCH] Better pipelining, round two

From: Daniel Stenberg <daniel_at_haxx.se>
Date: Wed, 16 Jan 2008 13:31:07 +0100 (CET)

On Wed, 16 Jan 2008, Dmitry Kurochkin wrote:

> Pipelining patch v3 attached.

Thanks! I committed it just now with some minor edits:

- library global symbols need to be Curl_-prefix by convention and I fixed a
   function

- static function are not Curl_-prefixed by convention so I cut that out from
   a few functions

- the info message when discivering a HTTP 1.1 server that supports pipelning
   was moved into a DEBUGF() macro since I believe very few users who simply
   use the library will care. The macro will include the code when built as
   with configure --enable-debug.

> Now I plan to try some real world usage with pipelining. I want to use
> it in darcs. But this will take some time.

Great! Also, if you feel brave and want to dig further into the lovely wonders
of libcurl and pipelining, I have this outstanding bug report from June
nagging on my conscience:

         http://curl.haxx.se/mail/lib-2007-06/0336.html

> BTW If there are any docs that should be updated I am ready to take over
> this.

There really isn't any pipelning docs yet for libcurl that I can think of, but
I figure it would make a sense to perhaps add a new section in the
libcurl-multi.3 man page that could descripe the general usage and what
libcurl does (and doesn't).

Oh, right, there's the lib/README.pipelining document that could be looked
over. But ideally we should rather push docs into the man pages instead of
those obscure README.*-files that hardly anyone will ever read...

-- 
  Commercial curl and libcurl Technical Support: http://haxx.se/curl.html
Received on 2008-01-16