]> git.ipfire.org Git - thirdparty/curl.git/commitdiff
multi: remove a few superfluous assigns
authorDaniel Stenberg <daniel@haxx.se>
Tue, 11 Apr 2023 13:01:22 +0000 (15:01 +0200)
committerDaniel Stenberg <daniel@haxx.se>
Tue, 11 Apr 2023 21:51:49 +0000 (23:51 +0200)
PVS found these "The 'rc' variable was assigned the same value." cases.

Ref: #10929
Closes #10932

lib/multi.c

index a011a36be4a3aae445bbd9612f3899a08a22c739..b2b1d65a38181711fbd17f55e7fc03a4c392a1cf 100644 (file)
@@ -2222,7 +2222,6 @@ static CURLMcode multi_runsingle(struct Curl_multi *multi,
             /* DO was not completed in one function call, we must continue
                DOING... */
             multistate(data, MSTATE_DOING);
-            rc = CURLM_OK;
           }
 
           /* after DO, go DO_DONE... or DO_MORE */
@@ -2230,7 +2229,6 @@ static CURLMcode multi_runsingle(struct Curl_multi *multi,
             /* we're supposed to do more, but we need to sit down, relax
                and wait a little while first */
             multistate(data, MSTATE_DOING_MORE);
-            rc = CURLM_OK;
           }
           else {
             /* we're done with the DO, now DID */
@@ -2331,9 +2329,8 @@ static CURLMcode multi_runsingle(struct Curl_multi *multi,
                      MSTATE_DID : MSTATE_DOING);
           rc = CURLM_CALL_MULTI_PERFORM;
         }
-        else
-          /* stay in DO_MORE */
-          rc = CURLM_OK;
+        /* else
+           stay in DO_MORE */
       }
       else {
         /* failure detected */
@@ -2562,7 +2559,6 @@ static CURLMcode multi_runsingle(struct Curl_multi *multi,
            won't get stuck on this transfer at the expense of other concurrent
            transfers */
         Curl_expire(data, 0, EXPIRE_RUN_NOW);
-        rc = CURLM_OK;
       }
       break;
     }