S390: Fix build error in iconvdata/bug-iconv11.c
Commit Message
Hi,
I get the following build error on S390 31bit while building the test
iconvdata/bug-iconv11.c with gcc 5.3:
gcc bug-iconv11.c -c -std=gnu11 -fgnu89-inline -O3 -Wall -Werror
-Wundef -Wwrite-strings -fmerge-all-constants -frounding-math
-Wstrict-prototypes -Wold-style-definition -mlong-double-128
-I../include -I/home/stli/glibcDir/glibc-20160119_31-build/iconvdata
-I/home/stli/glibcDir/glibc-20160119_31-build
-I../sysdeps/unix/sysv/linux/s390/s390-32
-I../sysdeps/unix/sysv/linux/s390/fpu -I../sysdeps/s390/fpu
-I../sysdeps/unix/sysv/linux/s390 -I../sysdeps/s390/nptl
-I../sysdeps/ieee754/ldbl-64-128 -I../sysdeps/ieee754/ldbl-opt
-I../sysdeps/unix/sysv/linux/include -I../sysdeps/unix/sysv/linux
-I../sysdeps/nptl -I../sysdeps/pthread -I../sysdeps/gnu
-I../sysdeps/unix/inet -I../sysdeps/unix/sysv -I../sysdeps/unix
-I../sysdeps/posix -I../sysdeps/s390/s390-32/multiarch
-I../sysdeps/s390/s390-32 -I../sysdeps/wordsize-32
-I../sysdeps/s390/multiarch -I../sysdeps/s390
-I../sysdeps/ieee754/ldbl-128 -I../sysdeps/ieee754/dbl-64
-I../sysdeps/ieee754/flt-32 -I../sysdeps/ieee754 -I../sysdeps/generic
-I.. -I../libio -I. -D_LIBC_REENTRANT -include
/home/stli/glibcDir/glibc-20160119_31-build/libc-modules.h
-DMODULE_NAME=nonlib -include ../include/libc-symbols.h -o
/home/stli/glibcDir/glibc-20160119_31-build/iconvdata/bug-iconv11.o -MD
-MP -MF
/home/stli/glibcDir/glibc-20160119_31-build/iconvdata/bug-iconv11.o.dt
-MT /home/stli/glibcDir/glibc-20160119_31-build/iconvdata/bug-iconv11.o
bug-iconv11.c: In function ‘test_ibm93x’:
bug-iconv11.c:59:11: error: format ‘%td’ expects argument of type
‘ptrdiff_t’, but argument 2 has type ‘size_t {aka long unsigned int}’
[-Werror=format=]
printf (" ==> %td: %s\n"
^
cc1: all warnings being treated as errors
This patch uses %zu format specifier for argument size_t ret instead of %td.
Ok to commit?
Bye
Stefan
ChangeLog:
* iconvdata/bug-iconv11.c (test_ibm93x):
Use %zu printf format specifier for size_t argument.
Comments
On Tue, 19 Jan 2016, Stefan Liebler wrote:
> This patch uses %zu format specifier for argument size_t ret instead of %td.
>
> Ok to commit?
OK.
@@ -56,7 +56,7 @@ test_ibm93x (const char *from_set, const char *input, size_t inbytesleft)
errno = 0;
size_t ret = iconv (cd, &inbuf, &inbytesleft, &outbuf, &outbytesleft);
- printf (" ==> %td: %s\n"
+ printf (" ==> %zu: %s\n"
" inbuf%+td, inbytesleft=%zu, outbuf%+td, outbytesleft=%zu\n",
ret, strerror (errno),
inbuf - input, inbytesleft, outbuf - output, outbytesleft);