On Tue, Jul 11, 2006 at 12:20:03PM -0400, Garrett Rooney wrote:
> On 7/10/06, Tyler MacDonald <[EMAIL PROTECTED]> 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.
joe
