[pacman-dev] [PATCH] Implement multiplexed download using mCURL

[Allan McRae]

Changes look good.  My only remaining comment is this:

On 27/3/20 6:59 am, Anatol Pomozov wrote:
>>> + * Returns -1 if an error happened
>>> + */
>>> +static int curl_multi_handle_single_done(CURLM *curlm, CURLMsg *msg, const char *localpath)
>> I have no idea what this function does from its name.
> This is a callback for mCURL event CURLMSG_DONE
> https://curl.haxx.se/libcurl/c/curl_multi_info_read.html
> We receive this event when one "curl easy" (or single) transfer is finished.
> 

To be clear, I can see what the function does from the function, but the
name is very vague.  If I saw "handle single done", I would not be any
the wiser what is going on.

How about:

curl_multi_check_finished_download()
curl_multi_download_completion_check()

or even:

curl_multi_handle_curlmsg_done()

Allan