Bruno Haible <[email protected]> writes: > Collin Funk wrote: >> Grisha Levit <[email protected]> writes: >> >> > There is code [1] in gnulib that disables this warning for config.h but >> > it is meant to be temporary and once removed would cause this snippet to >> > trigger a failure. >> > >> > [1] >> > https://cgit.git.savannah.gnu.org/cgit/gnulib.git/commit/?id=ff19eeafbee >> >> Was it supposed to be temporary? > > The comment says it: > > dnl This is a temporary workaround until Autoconf fixes it. > dnl Test case: > dnl empty1=; empty2=; AC_DEFINE_UNQUOTED([FOO], [$empty1$empty2], [...]) > dnl should produce "#define FOO /**/", not "#define FOO ".
Ah, okay. My mistake... I figured it would have been mentioned in the commit message and looked over the comments. > IIRC, I have not reported it to the Autoconf mailing list. So, if you > want to do that, please do! It looks like _AC_DEFINE_Q needs to have an extra check for an empty string. Let me see if I can figure out m4 enough to add it, or I will report it if I cannot. Thanks, Collin
