Use "foo(void)" for definitions of functions with no parameters.

Standard practice in PostgreSQL is to use "foo(void)" instead of
"foo()", as the latter looks like an "old-style" function
declaration.  Similar changes were made in commits cdf4b9aff2,
0e72b9d440, 7069dbcc31, f1283ed6cc, 7b66e2c086, e95126cf04, and
9f7c527af3.

Author: Bertrand Drouvot <bertranddrouvot.pg@gmail.com>
Reviewed-by: Matthias van de Meent <boekewurm+postgres@gmail.com>
Discussion: https://postgr.es/m/aTBObQPg%2Bps5I7vl%40ip-10-97-1-34.eu-west-3.compute.internal
This commit is contained in:
Nathan Bossart 2025-12-03 10:54:37 -06:00
parent be25c77677
commit 9b05e2ec08
9 changed files with 11 additions and 11 deletions

View File

@ -123,7 +123,7 @@ missing_match(const void *key1, const void *key2, Size keysize)
} }
static void static void
init_missing_cache() init_missing_cache(void)
{ {
HASHCTL hash_ctl; HASHCTL hash_ctl;

View File

@ -488,7 +488,7 @@ systable_beginscan(Relation heapRelation,
* is declared. * is declared.
*/ */
static inline void static inline void
HandleConcurrentAbort() HandleConcurrentAbort(void)
{ {
if (TransactionIdIsValid(CheckXidAlive) && if (TransactionIdIsValid(CheckXidAlive) &&
!TransactionIdIsInProgress(CheckXidAlive) && !TransactionIdIsInProgress(CheckXidAlive) &&

View File

@ -640,7 +640,7 @@ pa_detach_all_error_mq(void)
* Check if there are any pending spooled messages. * Check if there are any pending spooled messages.
*/ */
static bool static bool
pa_has_spooled_message_pending() pa_has_spooled_message_pending(void)
{ {
PartialFileSetState fileset_state; PartialFileSetState fileset_state;

View File

@ -711,7 +711,7 @@ LogicalRepSyncSequences(void)
* resource error and are not repeatable. * resource error and are not repeatable.
*/ */
static void static void
start_sequence_sync() start_sequence_sync(void)
{ {
Assert(am_sequencesync_worker()); Assert(am_sequencesync_worker());

View File

@ -1410,7 +1410,7 @@ check_and_set_sync_info(pid_t worker_pid)
* Reset syncing flag. * Reset syncing flag.
*/ */
static void static void
reset_syncing_flag() reset_syncing_flag(void)
{ {
SpinLockAcquire(&SlotSyncCtx->mutex); SpinLockAcquire(&SlotSyncCtx->mutex);
SlotSyncCtx->syncing = false; SlotSyncCtx->syncing = false;

View File

@ -1551,7 +1551,7 @@ start_table_sync(XLogRecPtr *origin_startpos, char **slotname)
* and starts streaming to catchup with apply worker. * and starts streaming to catchup with apply worker.
*/ */
static void static void
run_tablesync_worker() run_tablesync_worker(void)
{ {
char originname[NAMEDATALEN]; char originname[NAMEDATALEN];
XLogRecPtr origin_startpos = InvalidXLogRecPtr; XLogRecPtr origin_startpos = InvalidXLogRecPtr;

View File

@ -5561,7 +5561,7 @@ set_stream_options(WalRcvStreamOptions *options,
* Cleanup the memory for subxacts and reset the related variables. * Cleanup the memory for subxacts and reset the related variables.
*/ */
static inline void static inline void
cleanup_subxact_info() cleanup_subxact_info(void)
{ {
if (subxact_data.subxacts) if (subxact_data.subxacts)
pfree(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. * It sets up replication origin, streaming options and then starts streaming.
*/ */
static void static void
run_apply_worker() run_apply_worker(void)
{ {
char originname[NAMEDATALEN]; char originname[NAMEDATALEN];
XLogRecPtr origin_startpos = InvalidXLogRecPtr; XLogRecPtr origin_startpos = InvalidXLogRecPtr;

View File

@ -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 bool uuid_abbrev_abort(int memtupcount, SortSupport ssup);
static Datum uuid_abbrev_convert(Datum original, 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 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); static pg_uuid_t *generate_uuidv7(uint64 unix_ts_ms, uint32 sub_ms);
Datum Datum
@ -545,7 +545,7 @@ gen_random_uuid(PG_FUNCTION_ARGS)
* than the previous returned timestamp (on this backend). * than the previous returned timestamp (on this backend).
*/ */
static inline int64 static inline int64
get_real_time_ns_ascending() get_real_time_ns_ascending(void)
{ {
static int64 previous_ns = 0; static int64 previous_ns = 0;
int64 ns; int64 ns;

View File

@ -78,7 +78,7 @@ struct LogicalRepInfos
}; };
static void cleanup_objects_atexit(void); 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_base_conninfo(const char *conninfo, char **dbname);
static char *get_sub_conninfo(const struct CreateSubscriberOptions *opt); static char *get_sub_conninfo(const struct CreateSubscriberOptions *opt);
static char *get_exec_path(const char *argv0, const char *progname); static char *get_exec_path(const char *argv0, const char *progname);