Dan Fandrich wrote:
> On Tue, Jan 30, 2007 at 10:45:58AM +0100, Michael Wallner wrote:
>> attached is a patch against current CVS, which is supposed to add timeout
>> support in milliseconds (CURLOPT_TIMEOUT_MS, CURLOPT_CONNECTTIMEOUT_MS).
>>
>> AFAICT, there are three issues left:
>> - SIGALRM, alarm() accepts seconds only
>> - CONNECT, I lowered the timeout to 10ms

> Reducing the timeout to 10 ms here turns this practically into a busy loop,
> checking 100 times a second, which could be significant on slower platforms.

Yeah, that's why I listed it as remaining issues and placed a FIXME.

> Why can't the timeout value here be set to the value of 'check' and
> avoid the tight loop altogether?

No idea, CVS blame/log might give one.

Thanks,

-- 
Michael