Fix C23 compiler warning
authorPeter Eisentraut <peter@eisentraut.org>
Tue, 22 Oct 2024 06:12:43 +0000 (08:12 +0200)
committerPeter Eisentraut <peter@eisentraut.org>
Tue, 26 Nov 2024 12:36:49 +0000 (13:36 +0100)
commitf3834c12afd8125c4e5e8774a4b941a58dab8f5b
treedaa590bbfe54b3aeb429207270111d215e44ed84
parentad89c8bda1f096a8592f1c27abec1e34e5f10408
Fix C23 compiler warning

The approach of declaring a function pointer with an empty argument
list and hoping that the compiler will not complain about casting it
to another type no longer works with C23, because foo() is now
equivalent to foo(void).

We don't need to do this here.  With a few struct forward declarations
we can supply a correct argument list without having to pull in
another header file.

(This is the only new warning with C23.  Together with the previous
fix a67a49648d9, this makes the whole code compile cleanly under C23.)

Reviewed-by: Tom Lane <tgl@sss.pgh.pa.us>
Discussion: https://www.postgresql.org/message-id/flat/95c6a9bf-d306-43d8-b880-664ef08f2944%40eisentraut.org
Discussion: https://www.postgresql.org/message-id/flat/87o72eo9iu.fsf%40gentoo.org
src/include/nodes/pathnodes.h