httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From r..@covalent.net
Subject Re: cvs commit: httpd-2.0/modules/generators mod_status.c
Date Fri, 16 Feb 2001 00:34:08 GMT

> > Hold on... You can't use either apr_interval_t or apr_interval_short_t for seconds.
> > 
> > apr_interval_t is USEC, guarenteed to hold some +/- 292271 years.
> > 
> > apr_interval_short_t is USEC, guarenteed to hold only +/- 35 minutes
> > 
> > If you want simply seconds, store them in an apr_int_t.
> 
> That is *exactly* what FirstBill did. He put them into an apr_uint32_t. Then
> Ryan went and monkeyed with it :-)
> 
> The question is why did Ryan have a problem with apr_uint32_t? The code sure
> looked fine to start with.

All I know is I had four warnings with the old code, one per
line.  Regardless, this is fixed now, commit coming soon.

Ryan

_______________________________________________________________________________
Ryan Bloom                        	rbb@apache.org
406 29th St.
San Francisco, CA 94131
-------------------------------------------------------------------------------


Mime
View raw message