Quoting fixes in GLIBCXX_ENABLE
Ralf Wildenhues
Ralf.Wildenhues@gmx.de
Thu Sep 30 21:16:00 GMT 2010
Hello,
* Andreas Schwab wrote on Thu, Sep 30, 2010 at 11:42:50AM CEST:
> * acinclude.m4 (GLIBCXX_ENABLE): Properly quote argument to
> AC_HELP_STRING. Double quote expansion of _g_help.
> (GLIBCXX_ENABLE_LIBSTDCXX_TIME): Double quote third argument of
> GLIBCXX_ENABLE.
> (GLIBCXX_ENABLE_CHEADERS): Likewise.
> (GLIBCXX_ENABLE_CLOCALE): Likewise.
> (GLIBCXX_ENABLE_ALLOCATOR): Likewise.
> (GLIBCXX_ENABLE_CSTDIO): Likewise.
> (GLIBCXX_ENABLE_SYMVERS): Likewise.
> * configure: Regenerate.
I can't approve this, and the patch looks right, but ...
> --- libstdc++/acinclude.m4 (revision 164746)
> +++ libstdc++/acinclude.m4 (working copy)
> @@ -1710,7 +1710,7 @@
> dnl Default is generic.
> dnl
> AC_DEFUN([GLIBCXX_ENABLE_CLOCALE], [
> - GLIBCXX_ENABLE(clocale,auto,[@<:@=MODEL@:>@],
> + GLIBCXX_ENABLE(clocale,auto,[[[=MODEL]]],
... why move away from quadrigraphs? They are the most robust spelling
wrt. quoting. Ah, ok, Autoconf 2.64 m4_text_wrap still treated them as
four not one characters, so alignment is wrong; but this has been fixed
in newer Autoconf. FWIW, I intend to propose moving GCC to 2.68 for
4.6.
Cheers,
Ralf
More information about the Libstdc++
mailing list