On Sat, 2002-10-12 at 20:26, Paul J. Reder wrote:
> Okay, this takes care of item 4 from the list below. Thanks Brian, saves
> me from having to do the commit. :)
> 
> What about the other 3? Should they be fixed by the change from
> apr_time_t to apr_int64_t? Apr_time_t is really apr_int64_t under
> the covers and I was seeing only the lower 32 bits being set when
> the variables were assigned "0" and "-1". The value was correctly
> set when it was assigned APR_DATE_BAD (which has an embedded cast)
> so it seems that 1-3 still need to be done.

If I remember correctly, the ANSI arithmetic conversion rules
should cause the 0 or -1 to be sign-extended to long long (or
whatever other integral type apr_int64_t is typedef'ed to).

Brian


Reply via email to