Changeset 7738

Timestamp:

Jan 17, 2009, 10:49:15 PM (13 years ago)

Author:

charles

Message:

(trunk libT) minor cleanup

File:

• trunk/libtransmission/web.c (modified) (2 diffs)

trunk/libtransmission/web.c

@@ -126 +126 @@
         }
 
-        curl_easy_setopt( easy, CURLOPT_DNS_CACHE_TIMEOUT, 3600L );
-        curl_easy_setopt( easy, CURLOPT_CONNECTTIMEOUT, 120L );
+        curl_easy_setopt( easy, CURLOPT_DNS_CACHE_TIMEOUT, 360L );
+        curl_easy_setopt( easy, CURLOPT_CONNECTTIMEOUT, 60L );
         curl_easy_setopt( easy, CURLOPT_FOLLOWLOCATION, 1L );
         curl_easy_setopt( easy, CURLOPT_FORBID_REUSE, 1L );
+        curl_easy_setopt( easy, CURLOPT_MAXREDIRS, 16L );
         curl_easy_setopt( easy, CURLOPT_NOSIGNAL, 1L );
         curl_easy_setopt( easy, CURLOPT_PRIVATE, task );
@@ -200 +201 @@
 remove_finished_tasks( tr_web * g )
 {
-    for( ;; )
+    CURL * easy;
+
+    do
     {
-        int ignored;
-        CURLMsg * msg = curl_multi_info_read( g->multi, &ignored );
-
-        if( msg == NULL )
-        {
-            break;
-        }
-        else if( ( msg->msg == CURLMSG_DONE ) && ( msg->easy_handle != NULL ) )
-        {
-            CURL * easy = msg->easy_handle;
+        CURLMsg * msg;
+        int msgs_left;
+
+        easy = NULL;
+        while(( msg = curl_multi_info_read( g->multi, &msgs_left ))) {
+            if( msg->msg == CURLMSG_DONE ) {
+                easy = msg->easy_handle;
+                break;
+            }
+        }
+
+        if( easy ) {
             long code;
             struct tr_web_task * task;
             curl_easy_getinfo( easy, CURLINFO_PRIVATE, (void*)&task );
             curl_easy_getinfo( easy, CURLINFO_RESPONSE_CODE, &code );
-            task_finish( task, code );
             curl_multi_remove_handle( g->multi, easy );
             curl_easy_cleanup( easy );
-        }
-    }
+            task_finish( task, code );
+        }
+    }
+    while ( easy );
 
     g->prev_running = g->still_running;