Re: Re: fnmatch.c includes alloca.h conditionally?
Derek Robert Price <[email protected]> Mon, 17 May 2004 15:55:30 -0400
| Newsgroups | gmane.comp.lib.gnulib.bugs,gmane.comp.version-control.cvs.bugs |
|---|---|
| Organization | Ximbiot <http://ximbiot.com> |
| Message-ID | <[email protected]> |
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
Paul Eggert wrote:
>>Thus it should be okay for vasnprintf to include alloca.h
>>unconditionally and rely on HAVE_ALLOCA being set correctly, whether
>>IN_LIBINTL or not.
>
>
>Yes, but only if the vasnprintf module wants to make alloca a
>prerequisite module. Bruno may prefer not having that dependency,
>in which case the #include <alloca.h> must be protected by some
>sort of #ifdef.
According to modules/vasnprintf, alloca is already a prerequisite of
vasnprintf.
Anyhow, this point is mostly moot for me anyhow. I was just trying to
help clean up vasnprintf, or thought I was. I believe I started arguing
this from a misconception of my own. vasnprintf works for me as it is
since CVS is not IN_LIBINTL and HAVE_ALLOCA is now properly defined
via alloca.m4.
Thanks,
Derek
- --
*8^)
Email: [email protected]
Get CVS support at <http://ximbiot.com>!
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.1 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org
iD8DBQFAqRixLD1OTBfyMaQRAgINAJwIhTnc7HT3U6qqQw4zop7nc5vvrQCdE45H
gZ81vAE6olI4hSEPO3w2fCk=
=fACi
-----END PGP SIGNATURE-----