Subject: Re: ares_reinit patch

Re: ares_reinit patch

From: Daniel Stenberg <daniel_at_haxx.se>
Date: Tue, 15 Feb 2011 09:37:00 +0100 (CET)

On Tue, 15 Feb 2011, Dima Tisnek wrote:

> Cross-posting this to curl list because I've got some comments here
> and no comments on c-ares list.

The libcurl list is still not the right place so I'll respond only on the
c-ares list.

> * adds an ares_reinit functon, which reinitializes the channel same
> way as ares_init.

Nit: you're not following the code style accurately. We don't do multiple
statements on the same code line for example.

> * patches ares_gethostbyname and ares_gethostbyaddr with automatic updates.

I think we need a new option to enable this. Or possibly one that instead
disables it for those who have a reason for not wanting this - primarily I
would think of performance reasons.

-- 
  / daniel.haxx.se
Received on 2011-02-15