cURL / Mailing Lists / curl-library / Single Mail

curl-library

Re: Stripped keep-alive header

From: Daniel Stenberg <daniel_at_haxx.se>
Date: Fri, 13 Jan 2006 14:32:28 +0100 (CET)

On Fri, 13 Jan 2006, Allen Chan wrote:

(from a previous mail)

> Btw, how do i send this in? What diff program do you use?

diff unified diff format. Use (GNU) 'diff -u'.

> curl_easy_getinfo(http_handle, CURLINFO_PORT, &port);

...

> curl_easy_setopt(http_handle, CURLOPT_LOCAL_PORT, 1000);

This is an obvious inconsistency. I suggest both are called *_LOCAL_PORT to
separate them better from other options named *_PORT when talking about a
remote port number.

> I would like to hear any suggestions on how this particular effort should
> move forward. Perhaps feature requests? I'm out of ideas after
> CURLOPT_LOCAL_PORT.

I suggest you post a patch as a next step. Then we can offer feedback and
comments on the approach you've taken so far.

> Maybe once I get going on my project I'll come across something else lacking
> in curl.

Yeah, that's usually what happens when you start digging into things... :-)

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