[tlbuild] lcdf-typetools: buggy inline handling in config.h.in

[Eddie Kohler]

I'll fix this in the next version by splitting config.h.

Eddie


On 06/06/2010 02:27 PM, Vladimir Volovich wrote:
> Hi Eddie and Peter,
>
> in config.h.in, there's this snippet at the end of file:
>
> /* Get rid of a possible inline macro under C++. */
> # define inline inline
>
> (note that there's another snippet which deals with "inline" above this one).
>
> the problem is that when config.h is generated by config.status, it does
> change the above to:
>
> /* Get rid of a possible inline macro under C++. */
> # define inline __inline
>
> when "inline" was defined to something other than "inline" or nothing (e.g.
> my config.status on AIX has
>
> D["inline"]=" __inline"
>
> which results in the above expansion.
>
> (apparently, config.status touches not only lines which start with #undef
> but also lines with start with #define.)
>
> this results in unwanted definition of inline as a macro when compiling
> C++ files, and a build failure:
>
> gmake[5]: Entering directory `/usr/local/vvv/src/source/Work/texk/lcdf-typetools/liblcdf'
> source='../../../../texk/lcdf-typetools/liblcdf/bezier.cc' object='bezier.o' libtool=no \
>          DEPDIR=.deps depmode=aix /bin/bash ../../../../texk/lcdf-typetools/../../build-aux/depcomp \
>          xlC  -DHAVE_CONFIG_H -I. -I../../../../texk/lcdf-typetools/liblcdf -I.. -I../../../../texk/lcdf-typetools/include  -I/opt/freeware/include -DUSE_STDLIB_WORKAROUNDS  -O2 -qmaxmem=262144 -qarch=com -qlanglvl=extended -c -o bezier.o ../../../../texk/lcdf-typetools/liblcdf/bezier.cc
> "/usr/vacpp/include/math.h", line 34.14: 1540-0040 (S) The text "double" is unexpected.  "__inline" may be undeclared or ambiguous.
> "/usr/vacpp/include/math.h", line 35.14: 1540-0040 (S) The text "double" is unexpected.  "__inline" may be undeclared or ambiguous.
> "/usr/vacpp/include/math.h", line 36.14: 1540-0040 (S) The text "double" is unexpected.  "__inline" may be undeclared or ambiguous.
>
> Best,
> v.