From 9eeb7d3ed5436145c0e68a2b3e2788af05f0042f Mon Sep 17 00:00:00 2001 From: Daniel Stenberg Date: Tue, 11 Apr 2023 15:01:22 +0200 Subject: [PATCH] multi: remove a few superfluous assigns PVS found these "The 'rc' variable was assigned the same value." cases. Ref: #10929 Closes #10932 --- lib/multi.c | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/lib/multi.c b/lib/multi.c index a011a36be4..b2b1d65a38 100644 --- a/lib/multi.c +++ b/lib/multi.c @@ -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; } -- 2.47.3