bug#79567: [PATCH] maint: pacify -Werror=trailing-whitespace

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