Allow to compile against gcc 15 (C23).
authorTatsuo Ishii <ishii@postgresql.org>
Tue, 9 Sep 2025 01:02:35 +0000 (10:02 +0900)
committerTatsuo Ishii <ishii@postgresql.org>
Tue, 9 Sep 2025 02:43:59 +0000 (11:43 +0900)
commite7068df01f8d6fcb869dcd1e1e8a8c879863c85a
tree4ca23f044315602c6a2b723465011a1babff13aa
parentf4d94b8d921a1764ae6e94069ad87b526dcc1e99
Allow to compile against gcc 15 (C23).

This commit includes multiple fixes to compile Pgpool-II in Fedora 42,
which uses gcc 15 (C23).

- Modify pool_type.h. "bool" is now standard in C99 and
  above. PostgreSQL decided to require C99 to compile it. So we follow
  the way, which is just including <stdbool.h>.  Also we define
  TRUE/FALSE to (bool) 1 and (bool) 0 respectively. They are used only
  in Windows build in PostgreSQL but we still use them in some
  places. Eventually we should replace it with true/false since we do
  not support Windows.

- It is now required that function pointer arguments matches the
  function prototype to be called. For example:
  static pid_t worker_fork_a_child(ProcessType type, void (*func) (), void *params);
  should be:
  static pid_t worker_fork_a_child(ProcessType type, void (*func) (void *), void *params);

  Also the prototype of pool_create_relcache() is changed,

- raw_expression_tree_walker() calls walker() in many places. Now
  callers of walker() should cast the first argument of it using (Node
  *). We replace the call:
return walker(((RangeVar *) node)->alias, context);
with:
return WALK(((RangeVar *) node)->alias, context);
where WALK is defined as:
    #define WALK(n,c) walker((Node *) (n), c)

- Note: we have lots of warnings regarding OpenSSL while compiling
  Pgpool-II in Fedora42. The version used in Fedora42:

  $ openssl -version
  OpenSSL 3.2.4 11 Feb 2025 (Library: OpenSSL 3.2.4 11 Feb 2025)

  The fix is not included in this commit. We need
  to look into it in the future.

Discussion: https://github.com/pgpool/pgpool2/issues/124
Backpatch-through: v4.6
src/include/main/health_check.h
src/include/pool.h
src/include/pool_type.h
src/include/utils/pool_relcache.h
src/include/utils/pool_select_walker.h
src/main/health_check.c
src/main/pgpool_main.c
src/rewrite/pool_timestamp.c
src/streaming_replication/pool_worker_child.c
src/test/regression/tests/010.rewrite_timestamp/timestamp/main.c
src/utils/pool_relcache.c