Remove long-dead support for platforms without sig_atomic_t.

C89 requires <signal.h> to define sig_atomic_t, and there is no evidence
in the buildfarm that any supported platforms don't comply.  Remove the
configure test to stop wasting build cycles on a purely historical issue.
(Once upon a time, we cared about supporting C89-compliant compilers on
machines with pre-C89 system headers, but that use-case has been dead for
quite a few years.)

I have some other fixes planned in this area, but let's start with this
to see if the buildfarm produces any surprising results.

diff --git a/configure b/configure
index 0bed81c2e4a42615c9ba8121121865f2d3895af6..0376a7ca9197cb860096fd14694adf98677f662d 100755 (executable)
--- a/configure
+++ b/configure
@@ -14151,20 +14151,6 @@ _ACEOF
 fi
 
 
-# We also check for sig_atomic_t, which *should* be defined per ANSI
-# C, but is missing on some old platforms.
-ac_fn_c_check_type "$LINENO" "sig_atomic_t" "ac_cv_type_sig_atomic_t" "#include <signal.h>
-"
-if test "x$ac_cv_type_sig_atomic_t" = xyes; then :
-
-cat >>confdefs.h <<_ACEOF
-#define HAVE_SIG_ATOMIC_T 1
-_ACEOF
-
-
-fi
-
-
 # Check for extensions offering the integer scalar type __int128.
 { $as_echo "$as_me:${as_lineno-$LINENO}: checking for __int128" >&5
 $as_echo_n "checking for __int128... " >&6; }

diff --git a/configure.in b/configure.in
index a28f9ddb2520ba40afae686ec7d742adf8bad07e..5f16530fca87f00e7c41c9070688aa9b42c1f631 100644 (file)
--- a/configure.in
+++ b/configure.in
@@ -1831,10 +1831,6 @@ AC_DEFINE_UNQUOTED(MAXIMUM_ALIGNOF, $MAX_ALIGNOF, [Define as the maximum alignme
 AC_CHECK_TYPES([int8, uint8, int64, uint64], [], [],
 [#include <stdio.h>])
 
-# We also check for sig_atomic_t, which *should* be defined per ANSI
-# C, but is missing on some old platforms.
-AC_CHECK_TYPES(sig_atomic_t, [], [], [#include <signal.h>])
-
 # Check for extensions offering the integer scalar type __int128.
 PGAC_TYPE_128BIT_INT
 

diff --git a/src/include/c.h b/src/include/c.h
index b719eb95e141e05c17ae2a4a1e897bbb6e3b732a..f5da4676c6fb60f71795c1ec5330342c7945f163 100644 (file)
--- a/src/include/c.h
+++ b/src/include/c.h
@@ -334,11 +334,6 @@ typedef unsigned PG_INT128_TYPE uint128;
 #define HAVE_INT64_TIMESTAMP
 #endif
 
-/* sig_atomic_t is required by ANSI C, but may be missing on old platforms */
-#ifndef HAVE_SIG_ATOMIC_T
-typedef int sig_atomic_t;
-#endif
-
 /*
  * Size
  *             Size of any memory resident object, as returned by sizeof.

diff --git a/src/include/pg_config.h.in b/src/include/pg_config.h.in
index 8873dccd1e2fa976ac1dd7acbd749ac3eb722f72..1dcc9a9ee7ab28137cd733649aeefd3653637d65 100644 (file)
--- a/src/include/pg_config.h.in
+++ b/src/include/pg_config.h.in
@@ -449,9 +449,6 @@
 /* Define to 1 if you have sigsetjmp(). */
 #undef HAVE_SIGSETJMP
 
-/* Define to 1 if the system has the type `sig_atomic_t'. */
-#undef HAVE_SIG_ATOMIC_T
-
 /* Define to 1 if you have the `snprintf' function. */
 #undef HAVE_SNPRINTF
 

diff --git a/src/include/pg_config.h.win32 b/src/include/pg_config.h.win32
index ad61392841ef9736ba4d21451efaffcc31c6bdbf..bf69ef5bdeb86a52c45e8c3833ad61bfd41501c6 100644 (file)
--- a/src/include/pg_config.h.win32
+++ b/src/include/pg_config.h.win32
@@ -322,9 +322,6 @@
 /* Define to 1 if you have sigsetjmp(). */
 /* #undef HAVE_SIGSETJMP */
 
-/* Define to 1 if the system has the type `sig_atomic_t'. */
-#define HAVE_SIG_ATOMIC_T 1
-
 /* Define to 1 if you have the `snprintf' function. */
 /* #undef HAVE_SNPRINTF */