Skip to content

Commit 0f19d0f

Browse files
committed
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.
1 parent 794e255 commit 0f19d0f

File tree

5 files changed

+0
-29
lines changed

5 files changed

+0
-29
lines changed

configure

Lines changed: 0 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -14151,20 +14151,6 @@ _ACEOF
1415114151
fi
1415214152

1415314153

14154-
# We also check for sig_atomic_t, which *should* be defined per ANSI
14155-
# C, but is missing on some old platforms.
14156-
ac_fn_c_check_type "$LINENO" "sig_atomic_t" "ac_cv_type_sig_atomic_t" "#include <signal.h>
14157-
"
14158-
if test "x$ac_cv_type_sig_atomic_t" = xyes; then :
14159-
14160-
cat >>confdefs.h <<_ACEOF
14161-
#define HAVE_SIG_ATOMIC_T 1
14162-
_ACEOF
14163-
14164-
14165-
fi
14166-
14167-
1416814154
# Check for extensions offering the integer scalar type __int128.
1416914155
{ $as_echo "$as_me:${as_lineno-$LINENO}: checking for __int128" >&5
1417014156
$as_echo_n "checking for __int128... " >&6; }

configure.in

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1831,10 +1831,6 @@ AC_DEFINE_UNQUOTED(MAXIMUM_ALIGNOF, $MAX_ALIGNOF, [Define as the maximum alignme
18311831
AC_CHECK_TYPES([int8, uint8, int64, uint64], [], [],
18321832
[#include <stdio.h>])
18331833

1834-
# We also check for sig_atomic_t, which *should* be defined per ANSI
1835-
# C, but is missing on some old platforms.
1836-
AC_CHECK_TYPES(sig_atomic_t, [], [], [#include <signal.h>])
1837-
18381834
# Check for extensions offering the integer scalar type __int128.
18391835
PGAC_TYPE_128BIT_INT
18401836

src/include/c.h

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -334,11 +334,6 @@ typedef unsigned PG_INT128_TYPE uint128;
334334
#define HAVE_INT64_TIMESTAMP
335335
#endif
336336

337-
/* sig_atomic_t is required by ANSI C, but may be missing on old platforms */
338-
#ifndef HAVE_SIG_ATOMIC_T
339-
typedef int sig_atomic_t;
340-
#endif
341-
342337
/*
343338
* Size
344339
* Size of any memory resident object, as returned by sizeof.

src/include/pg_config.h.in

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -449,9 +449,6 @@
449449
/* Define to 1 if you have sigsetjmp(). */
450450
#undef HAVE_SIGSETJMP
451451

452-
/* Define to 1 if the system has the type `sig_atomic_t'. */
453-
#undef HAVE_SIG_ATOMIC_T
454-
455452
/* Define to 1 if you have the `snprintf' function. */
456453
#undef HAVE_SNPRINTF
457454

src/include/pg_config.h.win32

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -322,9 +322,6 @@
322322
/* Define to 1 if you have sigsetjmp(). */
323323
/* #undef HAVE_SIGSETJMP */
324324

325-
/* Define to 1 if the system has the type `sig_atomic_t'. */
326-
#define HAVE_SIG_ATOMIC_T 1
327-
328325
/* Define to 1 if you have the `snprintf' function. */
329326
/* #undef HAVE_SNPRINTF */
330327

0 commit comments

Comments
 (0)
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy