At 01:06 PM 06/12/2002, Cliff Woolley wrote:
On 12 Jun 2002 [EMAIL PROTECTED] wrote:
> +#define APR_TIME_USEC(time) ((apr_int32_t)(time) %
APR_USEC_PER_SEC)
> +
> +#define APR_TIME_SEC(time) ((apr_int64_t)(time) /
APR_USEC_PER_SEC)
> +
> +#define APR_TIME_FROM_SEC(sec) ((apr_time_t)(sec) *
APR_USEC_PER_SEC)
> +
> +#define APR_TIME_MAKE(sec, usec) ((apr_time_t)(sec) *
APR_USEC_PER_SEC + usec)
Aren't these all missing a set of parens? In other words, I think the
cast should occur after the division/multiplication.
Also, the last one doesn't have parens around usec.
--
Greg Marr
[EMAIL PROTECTED]
"We thought you were dead."
"I was, but I'm better now." - Sheridan, "The Summoning"
