Re: _GL_INLINE_HEADER_BEGIN/END left behind

2013-08-28 Thread Paul Eggert
Eric Blake wrote: > Just using this file as an example; your preprocessor indentation > doesn't match the prevailing style of # always in column 1. That indentation style dates back to the good old days when it wasn't portable to put '#error' in column 1. gcc -Wtraditional still warns about this;

Re: _GL_INLINE_HEADER_BEGIN/END left behind

2013-08-28 Thread Eric Blake
On 08/28/2013 07:50 PM, Paul Eggert wrote: > Sure, that's easy enough. I pushed this: > > +++ b/lib/gl_oset.h > @@ -21,6 +21,9 @@ > #include > #include > > +#ifndef _GL_INLINE_HEADER_BEGIN > + #error "Please include config.h first." > +#endif > _GL_INLINE_HEADER_BEGIN > #ifndef GL_OSET_IN

Re: _GL_INLINE_HEADER_BEGIN/END left behind

2013-08-28 Thread Paul Eggert
Sure, that's easy enough. I pushed this: headers: check that _GL_INLINE_HEADER_BEGIN is defined Suggested by Bruce Korb in: http://lists.gnu.org/archive/html/bug-gnulib/2013-08/msg00070.html * doc/extern-inline.texi (extern inline): Suggest checking that _GL_INLINE_HEADER_BEGIN is defined. * lib/

Re: _GL_INLINE_HEADER_BEGIN/END left behind

2013-08-28 Thread Bruce Korb
On 08/28/13 15:50, Bruce Korb wrote: The compile error message: gcc -std=gnu99 -DHAVE_CONFIG_H -I. -I.. -I../lib -I../src -I../lib -I../libopts \ -I../libopts -g -O2 -MT pw-pw.o -MD -MP -MF .deps/pw-pw.Tpo \ -c -o pw-pw.o `test -f 'pw.c' || echo './'`pw.c In file included from pw.h:47:0

Re: _GL_INLINE_HEADER_BEGIN/END left behind

2013-08-28 Thread Eric Blake
On 08/28/2013 04:55 PM, Eric Blake wrote: > On 08/28/2013 04:50 PM, Bruce Korb wrote: >> The compile error message: >> >>> gcc -std=gnu99 -DHAVE_CONFIG_H -I. -I.. -I../lib -I../src -I../lib >>> -I../libopts \ >>> -I../libopts -g -O2 -MT pw-pw.o -MD -MP -MF .deps/pw-pw.Tpo \ >>> -c -o pw-pw.o

Re: _GL_INLINE_HEADER_BEGIN/END left behind

2013-08-28 Thread Eric Blake
On 08/28/2013 04:50 PM, Bruce Korb wrote: > The compile error message: > >> gcc -std=gnu99 -DHAVE_CONFIG_H -I. -I.. -I../lib -I../src -I../lib >> -I../libopts \ >> -I../libopts -g -O2 -MT pw-pw.o -MD -MP -MF .deps/pw-pw.Tpo \ >> -c -o pw-pw.o `test -f 'pw.c' || echo './'`pw.c >> In file inc

_GL_INLINE_HEADER_BEGIN/END left behind

2013-08-28 Thread Bruce Korb
The compile error message: gcc -std=gnu99 -DHAVE_CONFIG_H -I. -I.. -I../lib -I../src -I../lib -I../libopts \ -I../libopts -g -O2 -MT pw-pw.o -MD -MP -MF .deps/pw-pw.Tpo \ -c -o pw-pw.o `test -f 'pw.c' || echo './'`pw.c In file included from pw.h:47:0, from pw.c:20: ../li