apr-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jeff Trawick <traw...@attglobal.net>
Subject Re: [PATCH] hex format string for apr_uint64_t
Date Tue, 15 Apr 2003 20:15:32 GMT
Joe Orton wrote:
> On Tue, Apr 15, 2003 at 01:58:23PM -0400, Jeff Trawick wrote:
> ...
> 
>> #define APR_UINT64_T_FMT_LEN     3
>>+#define APR_UINT64_T_HEX_FMT     "llx"
>>+#define APR_UINT64_T_HEX_FMT_LEN 3
>> #define APR_TIME_T_FMT APR_INT64_T_FMT
> 
> 
> There's no need to add another _FMT_LEN #define, they are not really
> necessary, sizeof() is good for that kind of thing.

I don't like the presence of them either, but if we stop following the 
convention that we mysteriously arrived at then I'd like to see the 
bunch of them dropped with APR 1.0.


Mime
View raw message