apr-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Joe Orton <jor...@redhat.com>
Subject Re: "z" modifier for APR_SIZE_T_FMT?
Date Wed, 12 Jul 2006 08:30:42 GMT
On Tue, Jul 11, 2006 at 12:20:03PM -0400, Garrett Rooney wrote:
> On 7/10/06, Tyler MacDonald <tyler@yi.org> wrote:
> >I recently had a bug reported on my package, that it failed to compile 
> >under
> >s390. I guess this is because I was just using "%u" instead of
> >APR_SIZE_T_FMT to print the size, but I can't really be sure because I 
> >don't
> >have access to an s390 to compile myself. :)
> >
> >This led me to check up on what APR_SIZE_T_FMT is actually defined as. On 
> >my
> >system, it's:
> >
> >#define APR_SIZE_T_FMT "d"
> >
> >Is there any reason why the "z" modifier is not included here? It seems 
> >like
> >that is what printf(3) says we want to use for formatting size_t types.
> Off the top of my head I'm not sure, perhaps configure just isn't
> smart enough to use that if it's available?  I'm also not sure if the
> apr *printf code knows about that format code...

It doesn't, but there's no reason why it couldn't be added.  The 
APR_{S,}SIZE_T_FMT macros are completely redundant if the z modifier is 
supported, I don't know why that wasn't done to start with, it has meant 
a lot of ugly code in a lot of places.


View raw message