On Thu, 7 Jan 2010, Adrian Michel wrote:

> I applied the new patch and here are the results:

As I said, the new patch has no different functionality than the previous one.
The fact that you get different results with this and the previous one implies
that your testing method is unreliable.

> 1. intially the host name was in the hosts file and I was able to connect
> with both versions of the dll (w/ and w/o patch) - expected

Since the patch only modifies the function that waits for the *connect* to
complete it was totally expected, yes. The connect happens _after_ the name
has been resolved properly.

> 2. I removed the host name from hosts and I wasn't able to connect with
> either version (w/ and w/o patch) - expected. BTW, the version with the
> patch returned the error result much faster (almost immediately), while the
> version w/o the patch took a quite long time (around 20 seconds).

So it then resolved the name with DNS and failed to connect to that address
faster with the new approach. Can you find anything that would explain this
behavior? It seems very odd to me.

> 3. I added the host name to hosts, and now the version w/o patch connects
> fine, but the version with the new patch doesn't connect any more.

So you're saying that it worked first when you had it in hosts, and then
stopped working when you had removed the hosts entry and put it back again?
How on earth would libcurl know or care about if you've touched your hosts
file? This doesn't make sense at all.

> Now, no matter whether the hosts contains the host name pointing to the
> local machine or not, the version w/ the patch fails consistently, while the
> version w/o the patch works as expected (other than not being able to get
> the call to the progress callback).

Based on these shaky results I no longer have faith in your tests so I'm not
sure how to proceeed. I think you should stop just explaining what happens and
instead enable CURLOPT_VERBOSE and show us proper libcurl ouputs to describe
what happens or similar. I'm suspecting you suffer from problems elsewhere and
those problems fool you to draw wrong conclusions on these libcurl changes
we're trying out.

With this patch applied, do you see any problems when using (lib)curl against
other sites/URLs ?

-- 
  / daniel.haxx.se
-------------------------------------------------------------------
List admin: http://cool.haxx.se/list/listinfo/curl-library
Etiquette:  http://curl.haxx.se/mail/etiquette.html