cURL / Mailing Lists / curl-library / Single Mail

curl-library

Re: [PATCH v2] Add connection delay to Happy Eyeballs.

From: Björn Stenberg <bjorn_at_haxx.se>
Date: Sun, 3 Nov 2013 14:38:56 +0100

Steve Holme wrote:
> I saw that most of the auto builds have mostly come back to life overnight
> and as such noticed that some tests are showing the "Whut?" connection
> issue.
>
> As such I have pushed my take on the Curl_is_connected() fix part of the
> above mentioned patch.

Actually, a separate fix to the "Whut?" bug was already posted four days ago in 20131030204012.GA1552@giant (http://curl.haxx.se/mail/lib-2013-10/0268.html, although for some reason the mail archive doesn't include the attached patch).

Also I respectfully disagree that replacing a switch with an if-else-if-else-if sequence is simplifying. Multiple nested if/elses are significantly more complex to read than a switch.

-- 
Björn
-------------------------------------------------------------------
List admin: http://cool.haxx.se/list/listinfo/curl-library
Etiquette:  http://curl.haxx.se/mail/etiquette.html
Received on 2013-11-03