[pacman-dev] [PATCH] Implement multiplexed download using mCURL
allan at archlinux.org
Tue Apr 14 02:41:48 UTC 2020
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
> 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.
More information about the pacman-dev