autoconf-patches
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

status.m4: do not check for ac_delim


From: Stepan Kasal
Subject: status.m4: do not check for ac_delim
Date: Fri, 17 Mar 2006 20:19:44 +0100
User-agent: Mutt/1.4.1i

Hello,
  when Dan Manthey was writing his ingenious implementation of multiline
substitutions, my pesky comments "what if the user does AC_SUBST(ac_delim)"
has made him to include a special check for this.

I deleted that check now.  See the attached patch.

First, the check was buggy anyway: it contained s/ac_delim/ instead of
s/@ac_delim@/

Second, I think this check is making the code less readable.
It would be better to call
        AC_FATAL([[Do not AC_SUBST([ac_delim]).]])
directly from AC_SUBST.

But I think the check is not necessary at all; ac_* variables are internal,
so you should not be surprised that things explode when you touch them.

Have a nice day,
        Stepan

Attachment: autoconf-20060317-delim.patch
Description: Text document


reply via email to

[Prev in Thread] Current Thread [Next in Thread]