autoconf-patches
[Top][All Lists]
Advanced

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

Re: [RFC] improve autotest syntax checks


From: Eric Blake
Subject: Re: [RFC] improve autotest syntax checks
Date: Fri, 07 Mar 2008 22:41:12 -0700
User-agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.1.12) Gecko/20080213 Thunderbird/2.0.0.12 Mnenhy/0.7.5.666

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

According to Ralf Wildenhues on 3/7/2008 11:30 AM:

Hi Ralf, and thanks for the review,

| m4_init([cleanup]) and AS_INIT([cleanup]) must belong to the least
| intuitive interfaces that I've seen so far.  Why something named init
| receives a cleanup argument rather than the name of or the thing that
| it initializes, will not be understandable to users, and any apparent
| analogy to AC_INIT or AT_INIT is just borked.
|
| I don't have a good idea how to improve this, but I must confess that I
| don't like it much.  Maybe have a macro m4_init_wrap for this or so?

In other words, guarantee FIFO behavior for m4_wrap (even with earlier M4
versions) and add m4_init_wrap with LIFO behavior?  Sounds better than
requiring registering the wrapup hook during initialization.  I'll work on
that idea, and resubmit the series for review.

|
| Also, this patch series (and maybe the interface change) deserves a NEWS
| entry.

Sure thing.

|
|> diff --git a/tests/autotest.at b/tests/autotest.at
|> index 885ecca..5d0cec8 100644
|> --- a/tests/autotest.at
|> +++ b/tests/autotest.at
| [...]
|
|> +AT_CHECK_AT_SYNTAX([Multiple AT@&address@hidden,
|> +[[AT_INIT([suite, take one])
|
| You need to double-quote comma inside AT_INIT's argument.

Actually, I added a use of m4_expand inside AT_INIT so that I didn't have
to double-quote the comma.  But as that was in a syntax failure test, I
guess I should use a comma in a different test expected to compile cleanly
if I intend to make that behavior the rule.

- --
Don't work too hard, make some time for fun as well!

Eric Blake             address@hidden
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.5 (Cygwin)
Comment: Public key at home.comcast.net/~ericblake/eblake.gpg
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iD8DBQFH0ib384KuGfSFAYARAqNOAKCxwCIC5HjSmtyvd5X422o8P1mKVgCePAja
6XnU1JXYbNBz4n9VeOoXc0M=
=fwpp
-----END PGP SIGNATURE-----




reply via email to

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