On 2014/10/26 18:22, Richard W.M. Jones wrote: > It's just there to stop unreasonable timeouts or negative numbers. > 100000 s is 27 hours, and no webserver I know of would keep a > connection open that long. Possibly not even the IP stack. >
Yes, it is. But 26 hours is OK? I just think we should assure the timeout as reasonable range, absolutely 100000 is too big IMO. > What's the difference between defining a number at the top of the file > to be used once, and placing it exactly where it is used? Except the > former introduces long range dependencies into the code making it > harder to read and more fragile when changed. That's the purpose using macro. If this value is used only one place in the curl.c (or other c files) now and future, you are fine with it. :) Best regards, -Gonglei