
Re: [PATCH] warnings cleanup: size_t should be formatted %lu rather than %d
[Date Prev][Date Next] [Thread Prev][Thread Next]Re: [PATCH] warnings cleanup: size_t should be formatted %lu rather than %d
- From: Lukas Slebodnik <lslebodn@redhat.com>
- To: Tim Rice <tim@multitalents.net>
- Cc: nss-pam-ldapd-users@lists.arthurdejong.org
- Reply-to: nss-pam-ldapd-users@lists.arthurdejong.org
- Subject: Re: [PATCH] warnings cleanup: size_t should be formatted %lu rather than %d
- Date: Wed, 11 Mar 2015 19:59:54 +0100
On (11/03/15 11:47), Tim Rice wrote: >On Wed, 11 Mar 2015, Lukas Slebodnik wrote: > >| On (11/03/15 10:55), Patrick McLean wrote: >| >In several places the code uses a %d format to print a size_t variable. >| >On amd64 at least size_t is an unsigned long, so use %lu instead. >| That's not good approach. The size differs on 32-bit platforms (ix86) >| >| You should use "%zd" for size_t variable >| and "%zu" for ssize_t variable. > >Only if you do not care about portability. >Some platforms do not have the z modifier. > It is part of c99 standard @see section 7.21.6.1 draft[1]. (page 311) Which platform do you mean and what solution is more portable? "%lu" fix warning just on specific architectures. LS [1] http://www.open-std.org/jtc1/sc22/wg14/www/docs/n1570.pdf -- To unsubscribe send an email to nss-pam-ldapd-users-unsubscribe@lists.arthurdejong.org or see http://lists.arthurdejong.org/nss-pam-ldapd-users/
- [PATCH] warnings cleanup: size_t should be formatted %lu rather than %d,
Patrick McLean
- Re: [PATCH] warnings cleanup: size_t should be formatted %lu rather than %d,
Lukas Slebodnik
- Re: [PATCH] warnings cleanup: size_t should be formatted %lu rather than %d,
Tim Rice
- Re: [PATCH] warnings cleanup: size_t should be formatted %lu rather than %d, Lukas Slebodnik
- Re: [PATCH] warnings cleanup: size_t should be formatted %lu rather than %d,
Tim Rice
- Re: [PATCH] warnings cleanup: size_t should be formatted %lu rather than %d,
Lukas Slebodnik
- Prev by Date: Re: [PATCH] warnings cleanup: size_t should be formatted %lu rather than %d
- Next by Date: Re: [PATCH] warnings cleanup: size_t should be formatted %lu rather than %d
- Previous by thread: Re: [PATCH] warnings cleanup: size_t should be formatted %lu rather than %d
- Next by thread: Re: [PATCH] warnings cleanup: size_t should be formatted %lu rather than %d