PRIu64
tridge at samba.org
tridge at samba.org
Tue Jul 26 03:48:38 GMT 2005
Tim,
> Which C99 compatible platforms does it not work on? I was planning
> to check out how well this flew on the build farm but you reverted
> it before too long.
We have a autoconf test for whether the platform has a C99 snprintf
call. Lots of our supported platforms fail that test, in which case we
replace snprintf using lib/replace/snprintf.c
The code in lib/replace/snprintf.c does not support the PRIu64
extension.
So, this code would break all platforms that fail our current snprintf
configure test, plus on all platforms that don't have the extension.
> Yes although by this stage it's starting to be a lot of work just to fix
> a couple of warnings. (-:
perhaps the simplest change would be to add:
#ifndef PRIu64
#define PRIu64 "llu"
#endif
to lib/replace/replace.h ? That should allow us to use it on all
platforms, and should work with our replacement snprintf code (as it
will just see a %llu format).
Cheers, Tridge
More information about the samba-technical
mailing list