httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From <...@covalent.net>
Subject Re: [PATCH] APR_INT64_T_FMT addition and modified AB
Date Thu, 12 Apr 2001 14:55:23 GMT

> +if test "$ac_cv_sizeof_long_long" = "$ac_cv_sizeof_int"; then
> +    int64_t_fmt='#define APR_INT64_T_FMT "d"'
> +elif test "$ac_cv_sizeof_long_long" = "$ac_cv_sizeof_long"; then
> +    int64_t_fmt='#define APR_INT64_T_FMT "ld"'
> +else
> +    int64_t_fmt='#define APR_INT64_T_FMT "lld"'
> +fi
> +
>
> This is perpetuating the old problem... you need to back up and start by
> defining what _is_ an apr_int64_t... something we have never done
> correctly.  Your logic above is getting there, but we can't assume
> long_long on all platforms (I don't believe.)  Need to get apr_int64_t
> defined right, then proceed.

Take a second look.  We actually check for the size of a bunch of
different types of integers, and pick the right one.

Ryan

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


Mime
View raw message