Re: FYI: doc formatting, new tests

[Gary V. Vaughan]

Morning Eric,

Eric Blake wrote:
> I noticed some `a' vs. `an' errors, and silenced some overfull hbox
> warnings from texi2dvi.  Then I noticed that several of the commented
> examples were not consistent with existing examples, and that our
> testsuite didn't exercise maketemp.

Nice.  Thankyou!

> I'm not sure if it is worth trying to make the testsuite check that
> changeword works as advertised when m4 is configured with
> --enable-changeword, since CVS head dropped this experimental
> feature.

I would leave things as they are.

> I also found another POSIX incompatibility - Solaris m4 follows
> POSIX on maketemp, and replaces the trailing XXX with the process
> id without creating a file (meaning that multiple uses of maketemp
> on the same string give the same result), whereas we are
> treating it more like mkstemp and actually creating a secure
> file with a unique name each time.  Perhaps, for POSIX compliance,
> we should make maketemp match Solaris behavior, and add a new
> macro makestemp that does the current maketemp behavior.  Should
> we be documenting known incompatibilities with POSIX in the
> 1.4.x branch, or just wait for 2.0 to worry about them?

2 possible improvements:

 i) document it in the Compatibility node of the manual
ii) follow the POSIX semantics only when POSIXLY_CORRECT is set.

I think changing and/or adding functionality in a maintenance release to
a stable branch is ill advised.

> 2006-05-27  Eric Blake  <address@hidden>
> 
>       * doc/m4.texinfo: Fix usage of a vs. an.
>       (Loops, Include, Cleardiv, Patsubst, Format, M4exit): Kill
>       overfull hbox warnings.
>       (Inhibiting Invocation, Divert, Maketemp, M4exit): Add new tests.

Thanks for this, and the other patches you have been working on :-)

Cheers,
        Gary.
-- 
Gary V. Vaughan      ())_.  address@hidden,gnu.org}
Research Scientist   ( '/   http://blog.azazil.net
GNU Hacker           / )=   http://trac.azazil.net/projects/libtool
Technical Author   `(_~)_   http://sources.redhat.com/autobook

signature.asc
Description: OpenPGP digital signature