cURL / Mailing Lists / curl-library / Single Mail

curl-library

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

From: Björn Stenberg <bjorn_at_haxx.se>
Date: Tue, 5 Nov 2013 23:59:59 +0100

Steve Holme wrote:
> A couple of problems in the auto builds:
>
> * The Solaris 10 build is reporting "url.c, line 5581: warning: end-of-loop
> code not reached" - while the loop breaks out because of a break statement
> it would seem that this loop is no longer required.

You are right. The purpose of the loop was removed in the HE patch. I'm attaching a patch to remove it.

> * The OpenSuSE 12.1 build is reporting valgrind memory issues in test 583.
> Whilst we typically see "in use at exit: 656 bytes in 4 blocks" is there
> possibly something else connect wise that is making this test fail? [2]

This looks odd. I'll take a deeper look tomorrow.

-- 
Björn
Received on 2013-11-06