Re: finish the GNULIB_POSIXCHECK framework

2018-08-05 Thread Jim Meyering
On Sun, Aug 5, 2018 at 11:26 PM, Bruno Haible wrote: > With this framework in place, it is now possible to eliminate the > "checking whether ... is declared without a macro" checks when they > are not needed. > > 2018-08-05 Bruno Haible > > Optimize the "checking whether ... is declared

iconv-h, monetary, utime-h: fixes for 'posixcheck'

2018-08-05 Thread Bruno Haible
The modules iconv-h, monetary, utime-h install their header file only conditionally. But that condition needs to evaluate to true if module 'posixcheck' is in use. This series of patches fixes it. 2018-08-05 Bruno Haible utime-h: Generate header file when module 'posixcheck' is in use

Re: finish the GNULIB_POSIXCHECK framework

2018-08-05 Thread Bruno Haible
With this framework in place, it is now possible to eliminate the "checking whether ... is declared without a macro" checks when they are not needed. 2018-08-05 Bruno Haible Optimize the "checking whether ... is declared without a macro" checks. Suggested by Paul Eggert in

iconv-h: enable 'posixcheck' warnings

2018-08-05 Thread Bruno Haible
2018-08-05 Bruno Haible iconv-h: Enable 'posixcheck' warnings. * m4/iconv_h.m4 (gl_ICONV_H): Check for declarations of iconv and iconv_open. * lib/iconv.in.h (iconv_open, iconv): Use _GL_WARN_ON_USE. diff --git a/lib/iconv.in.h b/lib/iconv.in.h index 882e4cf..58

fix interaction between 'extern-inline' and 'warn-on-use' modules

2018-08-05 Thread Bruno Haible
When running the documented recipe for use of the 'posixcheck' module on GNU hello, I'm getting this link error: gcc -g -O2 -o hello src/hello.o ./lib/libhello.a ./lib/libhello.a(closeout.o): In function `rpl_environ': /media/develdata/devel/HELLO/hello/lib/unistd.h:890: multiple definition o