Re: inline -Werror=missing-declarations warnings

2012-12-18 Thread Z. Majeed
the check as it stands works only for gcc 4.6 and later because of #pragma GCC diagnostic push|pop --- On Mon, 12/17/12, Pádraig Brady p...@draigbrady.com wrote: On 12/17/2012 11:50 AM, Z. Majeed wrote: Building latest [coreutils] git source (= gnulib d245e6d)... ... I had to configure

Re: C locale *printf functions ?

2012-12-18 Thread Jim Meyering
Ben Pfaff wrote: Here is a revised version. It passes the included tests, so I'll wait 72 hours for comments and then commit it if no one objects. ... ChangeLog | 43 lib/c-asprintf.c | 35 +++ lib/c-snprintf.c

[PATCH] argp: better 'inline'

2012-12-18 Thread Paul Eggert
Use extern-inline module to declare extern inline functions. This avoids some bogus warning diagnostics. Problem discovered when modifying GNU tar to use the manywarnings module. * lib/argp.h, lib/argp-xinl.c (ARGP_EI) [!_LIBC]: * lib/argp-fmtstream.h, lib/argp-fs-xinl.c (ARGP_FS_EI) [!_LIBC]:

Re: C locale *printf functions ?

2012-12-18 Thread Ben Pfaff
On Tue, Dec 18, 2012 at 06:41:47PM +0100, Jim Meyering wrote: Ben Pfaff wrote: Here is a revised version. It passes the included tests, so I'll wait 72 hours for comments and then commit it if no one objects. That sounds great. I haven't reviewed all of that, but really like the idea and