Re: [PATCHv2] Improve use of select in http backend
From: Junio C Hamano <hidden>
Date: 2016-06-15 22:52:22
I re-read this thread once again, and my understanding of the current situation on these two patch series is the following. Please correct me if I am wrong. * This is not a regression fix, this is not a correctness fix, but it is a performance improvement; * Jeff gave an idea for improvement around the use of (rather, not having to use) data_received; and * Mika understood Jeff's suggestion, but was hesitant due to one potential issue around curl_multi_fdset() and asked Daniel's opinion, to which Daniel responded that the worrysome situation would not happen. It appears to me that the next step is for Mika to decide either (1) we go ahead with the original patch and leave the improvement for later, or (2) update the patch as Jeff suggested and we review it again. I can go either way, but whichever way you choose, I would want to see the patches properly signed-off. Thanks.