Kamil Dudka wrote:
On Tuesday 06 of October 2009 20:30:00 Rob Crittenden wrote:No, that is needed to ensure it works the same way cross-platform. It is an NSPR-ism.I don't think so. The timeouts are computed wrong, nobody has noticed it so far. Which platform does it need actually?
Wait. You are asking the wrong question.
How do you know the timeouts given to PR_Recv()/PR_Send() are correct? It simply can't be the constant associated with handle. I don't think this is the way how libcurl deals with the remainder of timeout.
This is closer. The problem isn't that the conversion is ugly, it is that it isn't doing the right thing. You are right about the computation, seconds vs milliseconds, but assuming I'm reading the comments in url.c right the timeout is supposed to apply to the whole request and not an individual operation within that request.
This is very out-of-whack and I'm wondering if the framework code changed and the nss portion was not updated.
So yes, looks like something should change but I'm not entirely sure what value should be passed to timeout for PR_Recv/PR_Send.
rob
smime.p7s
Description: S/MIME Cryptographic Signature
------------------------------------------------------------------- List admin: http://cool.haxx.se/list/listinfo/curl-library Etiquette: http://curl.haxx.se/mail/etiquette.html
