Re: reordering output messages from ENABLE_C99

[Phil Edwards <pedwards at disaster dot jaj dot com>]

[gcc-patches added]

Approved by Benjamin, applied to trunk and branch.


Phil

On Tue, May 29, 2001 at 08:11:33PM -0400, Phil Edwards wrote:
> 
> Would anybody be offended by the following patch?  It shuffles some of the
> AC_MSG_* statements in GLIBCPP_ENABLE_C99 so that they actually surround the
> test statements, rather than doing the tests, then displaying "checking"
> and the result all at once.  It lets me see where the pauses for checking
> actually come into play; call it a pet peeve of mine.  :-)
> 
> It also moves a block of tests themselves so that we don't have nested
> checking statements.  The semantics and results are unchanged.
> 
> Phil
> 
> 
> 
> Index: acinclude.m4
> ===================================================================
> RCS file: /home/pme/Repositories/GCC/gcc/libstdc++-v3/acinclude.m4,v
> retrieving revision 1.148
> diff -c -3 -p -r1.148 acinclude.m4
> *** acinclude.m4	2001/05/25 21:53:51	1.148
> --- acinclude.m4	2001/05/30 01:22:39
> *************** AC_DEFUN(GLIBCPP_ENABLE_C99, [dnl
> *** 1316,1321 ****
> --- 1316,1322 ----
>   
>     # Check for the existence of <math.h> functions used if C99 is enabled.
>     ac_c99_math=yes;
> +   AC_MSG_CHECKING([for ISO C99 support in <math.h>])
>     AC_TRY_COMPILE([#include <math.h>],[fpclassify(0.0);],, [ac_c99_math=no])
>     AC_TRY_COMPILE([#include <math.h>],[isfinite(0.0);],, [ac_c99_math=no])
>     AC_TRY_COMPILE([#include <math.h>],[isinf(0.0);],, [ac_c99_math=no])
> *************** AC_DEFUN(GLIBCPP_ENABLE_C99, [dnl
> *** 1331,1349 ****
>   	         [islessgreater(0.0,0.0);],, [ac_c99_math=no])
>     AC_TRY_COMPILE([#include <math.h>],
>   	         [isunordered(0.0,0.0);],, [ac_c99_math=no])
> -   AC_MSG_CHECKING([for ISO C99 support in <math.h>])
>     AC_MSG_RESULT($ac_c99_math)
>   
>     # Check for the existence in <stdlib.h> of lldiv_t, et. al.
>     ac_c99_stdlib=yes;
> -   AC_TRY_COMPILE([#include <stdlib.h>],
> - 	         [char* tmp; strtoll("gnu", &tmp, 10);],, [ac_c99_stdlib=no])
> -   AC_TRY_COMPILE([#include <stdlib.h>],
> - 	         [char* tmp; strtoull("gnu", &tmp, 10);],, [ac_c99_stdlib=no])
> -   AC_TRY_COMPILE([#include <stdlib.h>], [llabs(10);],, [ac_c99_stdlib=no])
> -   AC_TRY_COMPILE([#include <stdlib.h>], [lldiv(10,1);],, [ac_c99_stdlib=no])
> -   AC_TRY_COMPILE([#include <stdlib.h>], [atoll("10");],, [ac_c99_stdlib=no])
> - 	
>     AC_MSG_CHECKING([for lldiv_t declaration])
>     AC_CACHE_VAL(ac_c99_lldiv_t, [
>     AC_TRY_COMPILE([#include <stdlib.h>], 
> --- 1332,1341 ----
> *************** AC_DEFUN(GLIBCPP_ENABLE_C99, [dnl
> *** 1351,1371 ****
>                      [ac_c99_lldiv_t=yes], [ac_c99_lldiv_t=no])
>     ])
>     AC_MSG_RESULT($ac_c99_lldiv_t)
>     if test x"$ac_c99_lldiv_t" = x"no"; then
>       ac_c99_stdlib=no; 
>     fi; 
> -   AC_MSG_CHECKING([for ISO C99 support in <stdlib.h>])
>     AC_MSG_RESULT($ac_c99_stdlib)
>   
>     # Check for the existence of <wchar.h> functions used if C99 is enabled.
>     ac_c99_wchar=yes;
>     AC_TRY_COMPILE([#include <wchar.h>], 
>   	         [wcstold(L"10.0", NULL);],, [ac_c99_wchar=no])
>     AC_TRY_COMPILE([#include <wchar.h>], 
>   	         [wcstoll(L"10", NULL, 10);],, [ac_c99_wchar=no])
>     AC_TRY_COMPILE([#include <wchar.h>], 
>   	         [wcstoull(L"10", NULL, 10);],, [ac_c99_wchar=no])
> -   AC_MSG_CHECKING([for ISO C99 support in <wchar.h>])
>     AC_MSG_RESULT($ac_c99_wchar)
>   
>     AC_MSG_CHECKING([for enabled ISO C99 support])
> --- 1343,1371 ----
>                      [ac_c99_lldiv_t=yes], [ac_c99_lldiv_t=no])
>     ])
>     AC_MSG_RESULT($ac_c99_lldiv_t)
> + 
> +   AC_MSG_CHECKING([for ISO C99 support in <stdlib.h>])
> +   AC_TRY_COMPILE([#include <stdlib.h>],
> + 	         [char* tmp; strtoll("gnu", &tmp, 10);],, [ac_c99_stdlib=no])
> +   AC_TRY_COMPILE([#include <stdlib.h>],
> + 	         [char* tmp; strtoull("gnu", &tmp, 10);],, [ac_c99_stdlib=no])
> +   AC_TRY_COMPILE([#include <stdlib.h>], [llabs(10);],, [ac_c99_stdlib=no])
> +   AC_TRY_COMPILE([#include <stdlib.h>], [lldiv(10,1);],, [ac_c99_stdlib=no])
> +   AC_TRY_COMPILE([#include <stdlib.h>], [atoll("10");],, [ac_c99_stdlib=no])
>     if test x"$ac_c99_lldiv_t" = x"no"; then
>       ac_c99_stdlib=no; 
>     fi; 
>     AC_MSG_RESULT($ac_c99_stdlib)
>   
>     # Check for the existence of <wchar.h> functions used if C99 is enabled.
>     ac_c99_wchar=yes;
> +   AC_MSG_CHECKING([for ISO C99 support in <wchar.h>])
>     AC_TRY_COMPILE([#include <wchar.h>], 
>   	         [wcstold(L"10.0", NULL);],, [ac_c99_wchar=no])
>     AC_TRY_COMPILE([#include <wchar.h>], 
>   	         [wcstoll(L"10", NULL, 10);],, [ac_c99_wchar=no])
>     AC_TRY_COMPILE([#include <wchar.h>], 
>   	         [wcstoull(L"10", NULL, 10);],, [ac_c99_wchar=no])
>     AC_MSG_RESULT($ac_c99_wchar)
>   
>     AC_MSG_CHECKING([for enabled ISO C99 support])