Hi, On Wed, Dec 03, 2025 at 10:15:41AM -0500, Tom Lane wrote: > Matthias van de Meent <[email protected]> writes: > > I noticed the only changes here are for `static` definitions. Are we > > just more careful with normal functions, or does the compiler complain > > more easily about such "incomplete" definitions when they're in > > headers or need to be linked against? > > Some years ago we had a buildfarm animal that would complain about > this construct, so the tree used to be clean. Probably it's just > chance that these have only snuck into local functions.
Thank you both for looking at it! The buildfarm animal remark makes me think to check with -Wstrict-prototypes and -Wold-style-definition. I just did that and found two more (added in v2 attached) that the coccinelle script missed... Those new two (run_apply_worker() and usage()) are also static, so that's just chance. Regards, -- Bertrand Drouvot PostgreSQL Contributors Team RDS Open Source Databases Amazon Web Services: https://aws.amazon.com
>From 3da01ff159d2e88fad0fd781d514215254614fba Mon Sep 17 00:00:00 2001 From: Bertrand Drouvot <[email protected]> Date: Wed, 3 Dec 2025 13:25:24 +0000 Subject: [PATCH v2] Use func(void) for functions with no parameters In C standards till C17, func() means "unspecified parameters" while func(void) means "no parameters". The former disables compile time type checking and was marked obsolescent in C99. This commit replaces empty parameter lists with explicit void to enable proper type checking and eliminate possible undefined behavior if the function is called with parameters. This also prevents real bugs (API misuse for example). Note: C23 made func() and func(void) equivalent. Author: Bertrand Drouvot <[email protected]> Reviewed-by: Matthias van de Meent <[email protected]> Reviewed-by: Tom Lane <[email protected]> Discussion: https://www.postgresql.org/message-id/flat/aTBObQPg%2Bps5I7vl%40ip-10-97-1-34.eu-west-3.compute.internal --- src/backend/access/common/heaptuple.c | 2 +- src/backend/access/index/genam.c | 2 +- src/backend/replication/logical/applyparallelworker.c | 2 +- src/backend/replication/logical/sequencesync.c | 2 +- src/backend/replication/logical/slotsync.c | 2 +- src/backend/replication/logical/tablesync.c | 2 +- src/backend/replication/logical/worker.c | 4 ++-- src/backend/utils/adt/uuid.c | 4 ++-- src/bin/pg_basebackup/pg_createsubscriber.c | 2 +- 9 files changed, 11 insertions(+), 11 deletions(-) 7.4% src/backend/access/common/ 8.4% src/backend/access/index/ 49.6% src/backend/replication/logical/ 26.9% src/backend/utils/adt/ 7.4% src/bin/pg_basebackup/ diff --git a/src/backend/access/common/heaptuple.c b/src/backend/access/common/heaptuple.c index 74a52d87067..1967b047020 100644 --- a/src/backend/access/common/heaptuple.c +++ b/src/backend/access/common/heaptuple.c @@ -123,7 +123,7 @@ missing_match(const void *key1, const void *key2, Size keysize) } static void -init_missing_cache() +init_missing_cache(void) { HASHCTL hash_ctl; diff --git a/src/backend/access/index/genam.c b/src/backend/access/index/genam.c index 0cb27af1310..c96917085c2 100644 --- a/src/backend/access/index/genam.c +++ b/src/backend/access/index/genam.c @@ -488,7 +488,7 @@ systable_beginscan(Relation heapRelation, * is declared. */ static inline void -HandleConcurrentAbort() +HandleConcurrentAbort(void) { if (TransactionIdIsValid(CheckXidAlive) && !TransactionIdIsInProgress(CheckXidAlive) && diff --git a/src/backend/replication/logical/applyparallelworker.c b/src/backend/replication/logical/applyparallelworker.c index baa68c1ab6c..a4eb3962cb1 100644 --- a/src/backend/replication/logical/applyparallelworker.c +++ b/src/backend/replication/logical/applyparallelworker.c @@ -640,7 +640,7 @@ pa_detach_all_error_mq(void) * Check if there are any pending spooled messages. */ static bool -pa_has_spooled_message_pending() +pa_has_spooled_message_pending(void) { PartialFileSetState fileset_state; diff --git a/src/backend/replication/logical/sequencesync.c b/src/backend/replication/logical/sequencesync.c index e093e65e540..019e5ec6a7d 100644 --- a/src/backend/replication/logical/sequencesync.c +++ b/src/backend/replication/logical/sequencesync.c @@ -711,7 +711,7 @@ LogicalRepSyncSequences(void) * resource error and are not repeatable. */ static void -start_sequence_sync() +start_sequence_sync(void) { Assert(am_sequencesync_worker()); diff --git a/src/backend/replication/logical/slotsync.c b/src/backend/replication/logical/slotsync.c index 53c7d629239..31d7cb3ca77 100644 --- a/src/backend/replication/logical/slotsync.c +++ b/src/backend/replication/logical/slotsync.c @@ -1410,7 +1410,7 @@ check_and_set_sync_info(pid_t worker_pid) * Reset syncing flag. */ static void -reset_syncing_flag() +reset_syncing_flag(void) { SpinLockAcquire(&SlotSyncCtx->mutex); SlotSyncCtx->syncing = false; diff --git a/src/backend/replication/logical/tablesync.c b/src/backend/replication/logical/tablesync.c index fa8e3bf969a..6bb0cbeedad 100644 --- a/src/backend/replication/logical/tablesync.c +++ b/src/backend/replication/logical/tablesync.c @@ -1551,7 +1551,7 @@ start_table_sync(XLogRecPtr *origin_startpos, char **slotname) * and starts streaming to catchup with apply worker. */ static void -run_tablesync_worker() +run_tablesync_worker(void) { char originname[NAMEDATALEN]; XLogRecPtr origin_startpos = InvalidXLogRecPtr; diff --git a/src/backend/replication/logical/worker.c b/src/backend/replication/logical/worker.c index 93970c6af29..a46f8e0644b 100644 --- a/src/backend/replication/logical/worker.c +++ b/src/backend/replication/logical/worker.c @@ -5561,7 +5561,7 @@ set_stream_options(WalRcvStreamOptions *options, * Cleanup the memory for subxacts and reset the related variables. */ static inline void -cleanup_subxact_info() +cleanup_subxact_info(void) { if (subxact_data.subxacts) pfree(subxact_data.subxacts); @@ -5621,7 +5621,7 @@ start_apply(XLogRecPtr origin_startpos) * It sets up replication origin, streaming options and then starts streaming. */ static void -run_apply_worker() +run_apply_worker(void) { char originname[NAMEDATALEN]; XLogRecPtr origin_startpos = InvalidXLogRecPtr; diff --git a/src/backend/utils/adt/uuid.c b/src/backend/utils/adt/uuid.c index e5f27ff892b..5df35d7cacb 100644 --- a/src/backend/utils/adt/uuid.c +++ b/src/backend/utils/adt/uuid.c @@ -71,7 +71,7 @@ static int uuid_fast_cmp(Datum x, Datum y, SortSupport ssup); static bool uuid_abbrev_abort(int memtupcount, SortSupport ssup); static Datum uuid_abbrev_convert(Datum original, SortSupport ssup); static inline void uuid_set_version(pg_uuid_t *uuid, unsigned char version); -static inline int64 get_real_time_ns_ascending(); +static inline int64 get_real_time_ns_ascending(void); static pg_uuid_t *generate_uuidv7(uint64 unix_ts_ms, uint32 sub_ms); Datum @@ -545,7 +545,7 @@ gen_random_uuid(PG_FUNCTION_ARGS) * than the previous returned timestamp (on this backend). */ static inline int64 -get_real_time_ns_ascending() +get_real_time_ns_ascending(void) { static int64 previous_ns = 0; int64 ns; diff --git a/src/bin/pg_basebackup/pg_createsubscriber.c b/src/bin/pg_basebackup/pg_createsubscriber.c index cc4be5d6ef4..43dc6ff7693 100644 --- a/src/bin/pg_basebackup/pg_createsubscriber.c +++ b/src/bin/pg_basebackup/pg_createsubscriber.c @@ -78,7 +78,7 @@ struct LogicalRepInfos }; static void cleanup_objects_atexit(void); -static void usage(); +static void usage(void); static char *get_base_conninfo(const char *conninfo, char **dbname); static char *get_sub_conninfo(const struct CreateSubscriberOptions *opt); static char *get_exec_path(const char *argv0, const char *progname); -- 2.34.1
