This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: PATCH: Add --enable-__cxa_atexit
- From: Alexandre Oliva <aoliva at redhat dot com>
- To: "H . J . Lu" <hjl at lucon dot org>
- Cc: Jack Howarth <howarth at nitro dot med dot uc dot edu>, gcc-patches at gcc dot gnu dot org
- Date: 31 May 2002 19:43:07 -0300
- Subject: Re: PATCH: Add --enable-__cxa_atexit
- Organization: GCC Team, Red Hat
- References: <200205310627.CAA13821@nitro.msbb.uc.edu><200205311604.MAA14609@nitro.msbb.uc.edu><20020531104311.A25686@lucon.org>
On May 31, 2002, "H . J . Lu" <hjl@lucon.org> wrote:
> +AC_ARG_ENABLE(__cxa_atexit,
> +[ --enable-__cxa_atexit enable __cxa_atexit for C++],
> +[], [enable___cxa_atexit=no])
^^^^^^^^^^^^^^^^^^^^^^ this is redundant.
If you specify --disable-__cxa_atexit, enable___cxa_atexit will be set
to no. Besides, if it defaults to yes, you should document the
--disable flag, not the --enable one. I know you've withdrawn your
patch, but I thought I'd point this out anyway, for future configury
patches.
--
Alexandre Oliva Enjoy Guarana', see http://www.ic.unicamp.br/~oliva/
Red Hat GCC Developer aoliva@{cygnus.com, redhat.com}
CS PhD student at IC-Unicamp oliva@{lsd.ic.unicamp.br, gnu.org}
Free Software Evangelist Professional serial bug killer