httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "William A. Rowe, Jr." <wr...@rowe-clan.net>
Subject Re: cvs commit: httpd-2.0/modules/generators mod_status.c
Date Thu, 15 Feb 2001 23:09:38 GMT
From: <rbb@covalent.net>
Sent: Thursday, February 15, 2001 4:59 PM


> On Thu, 15 Feb 2001, Cliff Woolley wrote:
> > --- rbb@covalent.net wrote:
> > > Guys, we have a time value that doesn't fit as either of these.  Either we
> > > need ANOTHER time type, or we have done something wrong.
> > 
> > We need another type.  OtherBill's suggestion is fine by me.
> 
> Same here.
> 
> > > Please do not tell me:
> > > apr_time_t - apr_time_t != (apr_time_t | apr_interval_time_t)
> > 
> > Of course it does:
> > 
> > apr_time_t - apr_time_t == apr_interval_time_t
> > 
> > BUT
> > 
> > ((apr_time_t - apr_time_t)/APR_USEC_PER_SEC) != (apr_time_t | apr_interval_time_t)
> > 
> > Hence the problem.
> 
> Yeah, but we could fix that by just removing the /APR_USEC_PER_SEC in the
> generator function, and put it back in the show_times function.
> 
> Oh well.  Let's just add the new type and be done with it.

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.


Mime
View raw message