> From: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED] > > wrowe 2002/07/14 23:34:33 > > Modified: network_io/win32 sendrecv.c > Log: > The last transpostion of ->timeout into apr_time fixed a bug > identified > by TANAKA Koichi <[EMAIL PROTECTED]>, where we used our old ms > based timeout within the timeval structure usec member. > > This patch fixes the only exception to needing the timeout_ms. NOTICE > that we first test that *timeout* is positive, since only then will > timeout_ms contain anything worth using.
This is the exact kind of code that is likely to introduce bugs, and why we shouldn't be duplicating the timeout value inside of APR. At some point, somebody is going to commit code to APR on Windows, that doesn't use both values. Ryan
