Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Commit0f19d0f

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 evidencein the buildfarm that any supported platforms don't comply. Remove theconfigure test to stop wasting build cycles on a purely historical issue.(Once upon a time, we cared about supporting C89-compliant compilers onmachines with pre-C89 system headers, but that use-case has been dead forquite a few years.)I have some other fixes planned in this area, but let's start with thisto see if the buildfarm produces any surprising results.
1 parent794e255 commit0f19d0f

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-
iftest"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
#defineHAVE_INT64_TIMESTAMP
335335
#endif
336336

337-
/* sig_atomic_t is required by ANSI C, but may be missing on old platforms */
338-
#ifndefHAVE_SIG_ATOMIC_T
339-
typedefintsig_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)

[8]ページ先頭

©2009-2025 Movatter.jp