Re: DNS servers on windows

2013-05-02 Thread Patrick Valsecchi
On 05/02/2013 10:55 PM, Daniel Stenberg wrote: On Thu, 2 May 2013, Patrick Valsecchi wrote: In attachment, you'll find my proposed patch for this problem. Where do we stand on this one? Do I have to change something in this proposed patch? Can you please provide a full patch, including a s

Re: Fixing a few errors and warning detected by coverity

2013-05-02 Thread Jaap Keuter
On 05/02/2013 10:51 PM, Daniel Stenberg wrote: > On Thu, 2 May 2013, Patrick Valsecchi wrote: > >> A colleague ran Coverity (a static analysis tool) against c-ares and fixed a >> few reported errors and warnings. >> >> Please have a look at the attached patch. > > Thanks! > > There's a large amo

Re: Cosmetic bugs in http://c-ares.haxx.se/ares_parse_a_reply.html and http://c-ares.haxx.se/ares_parse_aaaa_reply.html

2013-05-02 Thread Daniel Stenberg
On Thu, 21 Mar 2013, Alexander Klauer wrote: in the HTML pages mentioned in the subject, there are spurious '\fB' sequences in the synopses. Possibly they have to be changed to '\fP' in the source man pages, but I don't know enough roff to say for sure. Thanks! It took me forever but I've now

Re: DNS servers on windows

2013-05-02 Thread Daniel Stenberg
On Thu, 2 May 2013, Patrick Valsecchi wrote: In attachment, you'll find my proposed patch for this problem. Where do we stand on this one? Do I have to change something in this proposed patch? Can you please provide a full patch, including a sensible commit message that explains the change

Re: Fixing a few errors and warning detected by coverity

2013-05-02 Thread Daniel Stenberg
On Thu, 2 May 2013, Patrick Valsecchi wrote: A colleague ran Coverity (a static analysis tool) against c-ares and fixed a few reported errors and warnings. Please have a look at the attached patch. Thanks! There's a large amount of added typecasts for malloc() and a few other void pointers

Re: Patch to build current head (3f0ec47) on msvc11

2013-05-02 Thread Daniel Stenberg
On Wed, 1 May 2013, Alex Loukissas wrote: Since I figured out the issue, I thought I'd share the patch with you. Feel free to use it. Thanks, merged and pushed! -- / daniel.haxx.se