diff options
| author | Nathan Bossart <nathan@postgresql.org> | 2024-02-26 15:46:01 -0600 |
|---|---|---|
| committer | Nathan Bossart <nathan@postgresql.org> | 2024-02-26 15:46:01 -0600 |
| commit | 5fe08c006c826da4a7f5db2a79327477599edbc6 (patch) | |
| tree | 2f2e34f47f5d151fa3a9e1806a6d1c313bf3c859 /src | |
| parent | 6979ea2638a51c40acf6d04b816550b2c35b3e55 (diff) | |
| download | postgresql-5fe08c006c826da4a7f5db2a79327477599edbc6.tar.gz postgresql-5fe08c006c826da4a7f5db2a79327477599edbc6.zip | |
Use NULL instead of 0 for 'arg' argument in dshash_create() calls.
A couple of dshash_create() callers provide 0 for the 'void *arg'
argument, which might give readers the incorrect impression that
this is some sort of "flags" parameter.
Reviewed-by: Andy Fan
Discussion: https://postgr.es/m/20240119215941.GA1322079%40nathanxps13
Diffstat (limited to 'src')
| -rw-r--r-- | src/backend/replication/logical/launcher.c | 2 | ||||
| -rw-r--r-- | src/backend/utils/activity/pgstat_shmem.c | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/src/backend/replication/logical/launcher.c b/src/backend/replication/logical/launcher.c index 122db0bb134..ee66c292bd0 100644 --- a/src/backend/replication/logical/launcher.c +++ b/src/backend/replication/logical/launcher.c @@ -1013,7 +1013,7 @@ logicalrep_launcher_attach_dshmem(void) last_start_times_dsa = dsa_create(LWTRANCHE_LAUNCHER_DSA); dsa_pin(last_start_times_dsa); dsa_pin_mapping(last_start_times_dsa); - last_start_times = dshash_create(last_start_times_dsa, &dsh_params, 0); + last_start_times = dshash_create(last_start_times_dsa, &dsh_params, NULL); /* Store handles in shared memory for other backends to use. */ LogicalRepCtx->last_start_dsa = dsa_get_handle(last_start_times_dsa); diff --git a/src/backend/utils/activity/pgstat_shmem.c b/src/backend/utils/activity/pgstat_shmem.c index 3ce48e88a33..71410ddd3fb 100644 --- a/src/backend/utils/activity/pgstat_shmem.c +++ b/src/backend/utils/activity/pgstat_shmem.c @@ -180,7 +180,7 @@ StatsShmemInit(void) * With the limit in place, create the dshash table. XXX: It'd be nice * if there were dshash_create_in_place(). */ - dsh = dshash_create(dsa, &dsh_params, 0); + dsh = dshash_create(dsa, &dsh_params, NULL); ctl->hash_handle = dshash_get_hash_table_handle(dsh); /* lift limit set above */ |