multi-app.c example - missing call + question #8234
Comments
|
It does miss them, but that's actually fine and this code doesn't leak any memory. We should probably fix it though and not encourage that style. They are examples and I think its good that they work differently. They show concepts and that there's not just one way to it in an application. You decide what's best for your application. |
Fixes #8234 Reported-by: Melroy van den Berg
|
Maybe grep over the examples files. Since I can not promise this is the only example case with the multi API interface. Ps. Again regarding this |
What would the difference be? The API works as defined and documented. You can then select how to use those functions. This particular example chose to invoke the functions like this. |
Yes indeed, what would be the difference indeed. That was my question. But I think there is no difference, which is the answer I needed. Thank you. |
You are missing a
curl_multi_remove_handle()call in the multi-app.c example. Right?You could also clean-up the easy handles inside the curl_multi_info_read() method, correct?
Unrelated question: Sometimes I see the
while(..curl_multi_info_read())inside the mainwhile(still_running). But this multi-app.c example the two while loops are after each other.What is the best way to execute
curl_multi_info_read()? Inside the while loop of the other while (like crawler.c or 10-at-a-time.c examples), or after each other like the multi-app.c example?Thanks!
Regards,
Melroy van den Berg
The text was updated successfully, but these errors were encountered: