Don't use function definitions looking like old-style ones.

This fixes a bunch of somewhat pedantic warnings with new
compilers. Since by far the majority of other functions definitions use
the (void) style it just seems to be consistent to do so as well in the
remaining few places.
pull/7/head
Andres Freund 10 years ago
parent f9dec81a54
commit e95126cf04
  1. 2
      contrib/pg_standby/pg_standby.c
  2. 2
      src/backend/access/transam/twophase.c
  3. 2
      src/backend/commands/trigger.c
  4. 2
      src/backend/replication/logical/snapbuild.c
  5. 2
      src/backend/replication/repl_scanner.l
  6. 2
      src/backend/replication/walsender.c
  7. 2
      src/backend/storage/ipc/dsm_impl.c
  8. 2
      src/backend/utils/time/snapmgr.c
  9. 2
      src/bin/pg_resetxlog/pg_resetxlog.c
  10. 2
      src/pl/plperl/plperl.c

@ -171,7 +171,7 @@ CustomizableInitialize(void)
* Is the requested file ready yet? * Is the requested file ready yet?
*/ */
static bool static bool
CustomizableNextWALFileReady() CustomizableNextWALFileReady(void)
{ {
if (stat(WALFilePath, &stat_buf) == 0) if (stat(WALFilePath, &stat_buf) == 0)
{ {

@ -295,7 +295,7 @@ AtAbort_Twophase(void)
* PGXACT entry. * PGXACT entry.
*/ */
void void
PostPrepare_Twophase() PostPrepare_Twophase(void)
{ {
LWLockAcquire(TwoPhaseStateLock, LW_EXCLUSIVE); LWLockAcquire(TwoPhaseStateLock, LW_EXCLUSIVE);
MyLockedGxact->locking_backend = InvalidBackendId; MyLockedGxact->locking_backend = InvalidBackendId;

@ -3226,7 +3226,7 @@ static SetConstraintState SetConstraintStateAddItem(SetConstraintState state,
* Gets the current query fdw tuplestore and initializes it if necessary * Gets the current query fdw tuplestore and initializes it if necessary
*/ */
static Tuplestorestate * static Tuplestorestate *
GetCurrentFDWTuplestore() GetCurrentFDWTuplestore(void)
{ {
Tuplestorestate *ret; Tuplestorestate *ret;

@ -610,7 +610,7 @@ SnapBuildExportSnapshot(SnapBuild *builder)
* owner back to its original value. * owner back to its original value.
*/ */
void void
SnapBuildClearExportedSnapshot() SnapBuildClearExportedSnapshot(void)
{ {
/* nothing exported, thats the usual case */ /* nothing exported, thats the usual case */
if (!ExportInProgress) if (!ExportInProgress)

@ -237,7 +237,7 @@ replication_scanner_init(const char *str)
} }
void void
replication_scanner_finish() replication_scanner_finish(void)
{ {
yy_delete_buffer(scanbufhandle); yy_delete_buffer(scanbufhandle);
scanbufhandle = NULL; scanbufhandle = NULL;

@ -249,7 +249,7 @@ InitWalSender(void)
* process, similar to what transaction abort does in a regular backend. * process, similar to what transaction abort does in a regular backend.
*/ */
void void
WalSndErrorCleanup() WalSndErrorCleanup(void)
{ {
LWLockReleaseAll(); LWLockReleaseAll();

@ -1027,7 +1027,7 @@ dsm_impl_pin_segment(dsm_handle handle, void *impl_private)
} }
static int static int
errcode_for_dynamic_shared_memory() errcode_for_dynamic_shared_memory(void)
{ {
if (errno == EFBIG || errno == ENOMEM) if (errno == EFBIG || errno == ENOMEM)
return errcode(ERRCODE_OUT_OF_MEMORY); return errcode(ERRCODE_OUT_OF_MEMORY);

@ -345,7 +345,7 @@ GetNonHistoricCatalogSnapshot(Oid relid)
* cycles we spent tracking such fine details would be well-spent. * cycles we spent tracking such fine details would be well-spent.
*/ */
void void
InvalidateCatalogSnapshot() InvalidateCatalogSnapshot(void)
{ {
CatalogSnapshotStale = true; CatalogSnapshotStale = true;
} }

@ -703,7 +703,7 @@ PrintControlValues(bool guessed)
* Print the values to be changed. * Print the values to be changed.
*/ */
static void static void
PrintNewControlValues() PrintNewControlValues(void)
{ {
char fname[MAXFNAMELEN]; char fname[MAXFNAMELEN];

@ -2993,7 +2993,7 @@ plperl_hash_from_tuple(HeapTuple tuple, TupleDesc tupdesc)
static void static void
check_spi_usage_allowed() check_spi_usage_allowed(void)
{ {
/* see comment in plperl_fini() */ /* see comment in plperl_fini() */
if (plperl_ending) if (plperl_ending)

Loading…
Cancel
Save