> That's how it it was originally.  It was changed to this model not long
> after the original code was committed.  One of the problems with using
> seconds and a separate microsecond field, is that platforms other than
> Unix don't have the same reliance on seconds.  I believe Windows uses 100
> nanosecond blocks.

I realize that, but APR will be faster and considerably less bug-prone
on both platforms if the time structure contains one variable for
seconds and one for microseconds, regardless of how the time was
generated.  If you want to really optimize it for speed, include a third
variable for the native OS time_t, and a const char * for storing the
rfc822 date string the first time it is converted.

Regardless, APR and httpd have assumptions strewn throughout the code
that apr_time_t is a long long in microseconds, which isn't native
on any platform and makes every time operation slower for no reason.

....Roy

Reply via email to