[gnurl] 09/18: create_conn: prefer multiplexing to using new connections

[gnunet]

This is an automated email from the git hooks/post-receive script.

ng0 pushed a commit to branch master
in repository gnurl.

commit 8c0807aa161d791fa94da2e19e46b8cf845fd5d1
Author: Daniel Stenberg <address@hidden>
AuthorDate: Wed Dec 18 14:03:36 2019 +0100

    create_conn: prefer multiplexing to using new connections
    
    ... as it would previously prefer new connections rather than
    multiplexing in most conditions! The (now removed) code was a leftover
    from the Pipelining code that was translated wrongly into a
    multiplex-only world.
    
    Reported-by: Kunal Ekawde
    Bug: https://curl.haxx.se/mail/lib-2019-12/0060.html
    Closes #4732
---
 lib/url.c | 18 ------------------
 1 file changed, 18 deletions(-)

diff --git a/lib/url.c b/lib/url.c
index 4111eec3a..56fb73636 100644
--- a/lib/url.c
+++ b/lib/url.c
@@ -3594,24 +3594,6 @@ static CURLcode create_conn(struct Curl_easy *data,
   else
     reuse = ConnectionExists(data, conn, &conn_temp, &force_reuse, &waitpipe);
 
-  /* If we found a reusable connection that is now marked as in use, we may
-     still want to open a new connection if we are multiplexing. */
-  if(reuse && !force_reuse && IsMultiplexingPossible(data, conn_temp)) {
-    size_t multiplexed = CONN_INUSE(conn_temp);
-    if(multiplexed > 0) {
-      infof(data, "Found connection %ld, with %zu requests on it\n",
-            conn_temp->connection_id, multiplexed);
-
-      if(Curl_conncache_bundle_size(conn_temp) < max_host_connections &&
-         Curl_conncache_size(data) < max_total_connections) {
-        /* We want a new connection anyway */
-        reuse = FALSE;
-
-        infof(data, "We can reuse, but we want a new connection anyway\n");
-      }
-    }
-  }
-
   if(reuse) {
     /*
      * We already have a connection for this, we got the former connection

-- 
To stop receiving notification emails like this one, please contact
address@hidden.