Hi Ray I took the liberty of trying to make your patch apply.
https://github.com/Andersbakken/curl/tree/fix-curlopt_resolve I had to make some small changes: 1) There were a couple of lines with 4 spaces indentation level 2) Some comments were too long 3) Some for loops started with a space (for( ;...) I also had to make a change in Curl_loadhostpairs to store the information from the old handle before inserting the new one since the old sometimes was freed by the insertion. I am not 100% this was correct. (see 1c52fd75d068d4434bc7e0c96a46b0c8e2710f22) I made the decision to use your version of Curl_loadhostpairs wholesale rather than trying to mix your version of dns cache removals with the current one. Not sure if you have the time to or interesting in taking a look at this but I would be very grateful if you do. best regards Anders Bakken On Wed, May 27, 2015 at 4:10 PM, Ray Satiro via curl-library <[email protected]> wrote: > On 5/27/2015 6:51 PM, Anders Bakken via curl-library wrote: >> >> How much work do you think it would be? (and could you point me in the >> general right direction for it?) >> > > I worked on it a while ago. Stefan had some changes to add address remove > capability that I thought pre-empted mine so his were added instead with the > plan that mine would make it in at some later date. I have to adjust my code > to mesh with those changes. It's a low priority for me, it's been on the > backburner. I never tested happy eyeballs with my changes. > > http://curl.haxx.se/mail/lib-2015-03/0012.html > https://github.com/bagder/curl/compare/master...jay:fix-curlopt_resolve?expand=1 > > ------------------------------------------------------------------- > List admin: http://cool.haxx.se/list/listinfo/curl-library > Etiquette: http://curl.haxx.se/mail/etiquette.html -- Anders Bakken - Netflix ------------------------------------------------------------------- List admin: http://cool.haxx.se/list/listinfo/curl-library Etiquette: http://curl.haxx.se/mail/etiquette.html
