Subject: Re: yangtse: curl/ares ares__timeval.c,NONE,1.1

Re: yangtse: curl/ares ares__timeval.c,NONE,1.1

From: Yang Tse <yangsita_at_gmail.com>
Date: Sat, 10 May 2008 03:46:31 +0200

2008/5/9, Daniel Stenberg wrote:

> > Added Files:
> > ares__timeval.c
> > Log Message:
> > Use monotonic time source if available.
>
> Is anything using this file/these functions yet?

Nope, not yet.

> To me, it looks like my millisecond patch[*] might be easier to get working
> for windows using them...
>
> [*] =
> http://daniel.haxx.se/projects/c-ares/mail/c-ares-archive-2007-12/0007.shtml

Yes, that pending ms patch was the main reason for the introduction of
c-ares private functions ares__tvnow() and ares__tvdiff(). I started
looking at the ms patch and the need was obvious.

Before finally trying your ms second patch I'll have to complete this
one a little bit further with some autoconf magic. Some systems need
linking with librt to use clock_gettime, and I still have to address
this.

I won't be able to comment specific details of your ms patch in a
couple of days, but the feeling is certainly good.

-- 
-=[Yang]=-
Received on 2008-05-10