Re: [PATCH] Make AC_COMPUTE_INT public

[Stepan Kasal]

Hello,

On Wed, Jun 28, 2006 at 09:00:45PM +0200, Paolo Bonzini wrote:
> >The new version does not contain AC_LANG_COMPILER_REQUIRE()--is that

> I meant to remove it from _AC_COMPILE_INT (to parallel 
> _AC_COMPILE_IFELSE and _AC_RUN_IFELSE) and removed it from the wrong one.

I do not buy the parallel:  _AC_COMPILE_IFELSE and _AC_RUN_IFELSE are
used internally by Autoconf, and the fact that they do not call
AC_LANG_COMPILER_REQUIRE is part of their definition.

_AC_COMPILE_INT, OTOH, is doomed to deletion, sooner or later, and is
kept for compatibility.  This implies that we should not modify it.

BTW: perhaps we could add a FIXME to the description of it,
mentioning that we are going to delete it.

> Do you think it's better to leave it there in both macros?

It's the most natural thing to do, I guess.

Have a nice day,
        Stepan