diff --git a/contrib/btree_gist/btree_bit.c b/contrib/btree_gist/btree_bit.c index 2225244ded..5b246bcde4 100644 --- a/contrib/btree_gist/btree_bit.c +++ b/contrib/btree_gist/btree_bit.c @@ -104,7 +104,6 @@ gbt_bit_l2n(GBT_VARKEY *leaf, FmgrInfo *flinfo) pfree(o); return out; - } static const gbtree_vinfo tinfo = diff --git a/contrib/btree_gist/btree_cash.c b/contrib/btree_gist/btree_cash.c index dfa23224b6..6ac97e2b12 100644 --- a/contrib/btree_gist/btree_cash.c +++ b/contrib/btree_gist/btree_cash.c @@ -195,7 +195,6 @@ gbt_cash_penalty(PG_FUNCTION_ARGS) penalty_num(result, origentry->lower, origentry->upper, newentry->lower, newentry->upper); PG_RETURN_POINTER(result); - } Datum diff --git a/contrib/btree_gist/btree_float4.c b/contrib/btree_gist/btree_float4.c index b5d9175e6d..3cbd9a06dc 100644 --- a/contrib/btree_gist/btree_float4.c +++ b/contrib/btree_gist/btree_float4.c @@ -190,7 +190,6 @@ gbt_float4_penalty(PG_FUNCTION_ARGS) penalty_num(result, origentry->lower, origentry->upper, newentry->lower, newentry->upper); PG_RETURN_POINTER(result); - } Datum diff --git a/contrib/btree_gist/btree_float8.c b/contrib/btree_gist/btree_float8.c index 8fe1fedeef..b95a08e228 100644 --- a/contrib/btree_gist/btree_float8.c +++ b/contrib/btree_gist/btree_float8.c @@ -197,7 +197,6 @@ gbt_float8_penalty(PG_FUNCTION_ARGS) penalty_num(result, origentry->lower, origentry->upper, newentry->lower, newentry->upper); PG_RETURN_POINTER(result); - } Datum diff --git a/contrib/btree_gist/btree_inet.c b/contrib/btree_gist/btree_inet.c index e4b3a946b2..2fb952dca8 100644 --- a/contrib/btree_gist/btree_inet.c +++ b/contrib/btree_gist/btree_inet.c @@ -165,7 +165,6 @@ gbt_inet_penalty(PG_FUNCTION_ARGS) penalty_num(result, origentry->lower, origentry->upper, newentry->lower, newentry->upper); PG_RETURN_POINTER(result); - } Datum diff --git a/contrib/btree_gist/btree_interval.c b/contrib/btree_gist/btree_interval.c index a4b3b2b1e6..c2bf82086d 100644 --- a/contrib/btree_gist/btree_interval.c +++ b/contrib/btree_gist/btree_interval.c @@ -173,7 +173,6 @@ gbt_intv_compress(PG_FUNCTION_ARGS) } PG_RETURN_POINTER(retval); - } Datum @@ -276,7 +275,6 @@ gbt_intv_penalty(PG_FUNCTION_ARGS) penalty_num(result, iorg[0], iorg[1], inew[0], inew[1]); PG_RETURN_POINTER(result); - } Datum diff --git a/contrib/btree_gist/btree_macaddr.c b/contrib/btree_gist/btree_macaddr.c index 7f0e9e9c91..17290529c0 100644 --- a/contrib/btree_gist/btree_macaddr.c +++ b/contrib/btree_gist/btree_macaddr.c @@ -174,7 +174,6 @@ gbt_macad_penalty(PG_FUNCTION_ARGS) penalty_num(result, iorg[0], iorg[1], inew[0], inew[1]); PG_RETURN_POINTER(result); - } Datum diff --git a/contrib/btree_gist/btree_macaddr8.c b/contrib/btree_gist/btree_macaddr8.c index ab4bca5d50..796cc4efee 100644 --- a/contrib/btree_gist/btree_macaddr8.c +++ b/contrib/btree_gist/btree_macaddr8.c @@ -174,7 +174,6 @@ gbt_macad8_penalty(PG_FUNCTION_ARGS) penalty_num(result, iorg[0], iorg[1], inew[0], inew[1]); PG_RETURN_POINTER(result); - } Datum diff --git a/contrib/btree_gist/btree_ts.c b/contrib/btree_gist/btree_ts.c index 2671ba961c..a51900941e 100644 --- a/contrib/btree_gist/btree_ts.c +++ b/contrib/btree_gist/btree_ts.c @@ -377,7 +377,6 @@ gbt_ts_penalty(PG_FUNCTION_ARGS) penalty_num(result, orgdbl[0], orgdbl[1], newdbl[0], newdbl[1]); PG_RETURN_POINTER(result); - } diff --git a/contrib/dblink/dblink.c b/contrib/dblink/dblink.c index a06d4bd12d..a561d1d652 100644 --- a/contrib/dblink/dblink.c +++ b/contrib/dblink/dblink.c @@ -1004,7 +1004,6 @@ materializeResult(FunctionCallInfo fcinfo, PGconn *conn, PGresult *res) /* clean up GUC settings, if we changed any */ restoreLocalGucs(nestlevel); - } } PG_FINALLY(); @@ -2635,7 +2634,6 @@ deleteConnection(const char *name) ereport(ERROR, (errcode(ERRCODE_UNDEFINED_OBJECT), errmsg("undefined connection name"))); - } static void diff --git a/contrib/intarray/_int_bool.c b/contrib/intarray/_int_bool.c index 4b6a31080e..3ed88af76d 100644 --- a/contrib/intarray/_int_bool.c +++ b/contrib/intarray/_int_bool.c @@ -210,7 +210,6 @@ makepol(WORKSTATE *state) (errcode(ERRCODE_SYNTAX_ERROR), errmsg("syntax error"))); return ERR; - } } diff --git a/contrib/ltree/ltxtquery_io.c b/contrib/ltree/ltxtquery_io.c index d967f92110..3eca5cb8ff 100644 --- a/contrib/ltree/ltxtquery_io.c +++ b/contrib/ltree/ltxtquery_io.c @@ -260,7 +260,6 @@ makepol(QPRS_STATE *state) errmsg("syntax error"))); return ERR; - } } while (lenstack) diff --git a/contrib/pg_stat_statements/pg_stat_statements.c b/contrib/pg_stat_statements/pg_stat_statements.c index 1ca67ef623..df2ce63790 100644 --- a/contrib/pg_stat_statements/pg_stat_statements.c +++ b/contrib/pg_stat_statements/pg_stat_statements.c @@ -1849,7 +1849,6 @@ pg_stat_statements_internal(FunctionCallInfo fcinfo, if (qbuffer) free(qbuffer); - } /* Number of output arguments (columns) for pg_stat_statements_info */ diff --git a/contrib/postgres_fdw/connection.c b/contrib/postgres_fdw/connection.c index 129ca79221..f9b8c01f3b 100644 --- a/contrib/postgres_fdw/connection.c +++ b/contrib/postgres_fdw/connection.c @@ -1734,7 +1734,6 @@ postgres_fdw_get_connections(PG_FUNCTION_ARGS) tuplestore_putvalues(rsinfo->setResult, rsinfo->setDesc, values, nulls); } - PG_RETURN_VOID(); } diff --git a/contrib/spi/refint.c b/contrib/spi/refint.c index 6fbfef2b12..18062eb1cf 100644 --- a/contrib/spi/refint.c +++ b/contrib/spi/refint.c @@ -497,12 +497,10 @@ check_foreign_key(PG_FUNCTION_ARGS) nv, (is_char_type > 0) ? "'" : "", (k < nkeys) ? ", " : ""); } strcat(sql, " where "); - } else /* DELETE */ snprintf(sql, sizeof(sql), "delete from %s where ", relname); - } /* diff --git a/src/backend/access/brin/brin_bloom.c b/src/backend/access/brin/brin_bloom.c index 7428fa90d3..6812ca9fd1 100644 --- a/src/backend/access/brin/brin_bloom.c +++ b/src/backend/access/brin/brin_bloom.c @@ -257,7 +257,6 @@ typedef struct BloomFilter /* data of the bloom filter */ char data[FLEXIBLE_ARRAY_MEMBER]; - } BloomFilter; diff --git a/src/backend/access/gin/ginbtree.c b/src/backend/access/gin/ginbtree.c index 8df45478f1..cc6d4e6338 100644 --- a/src/backend/access/gin/ginbtree.c +++ b/src/backend/access/gin/ginbtree.c @@ -527,7 +527,6 @@ ginPlaceToPage(GinBtree btree, GinBtreeStack *stack, BufferGetBlockNumber(stack->buffer), BufferGetBlockNumber(rbuffer)); } - } else { diff --git a/src/backend/access/heap/heapam.c b/src/backend/access/heap/heapam.c index ba11bcd99e..8d7655f4cf 100644 --- a/src/backend/access/heap/heapam.c +++ b/src/backend/access/heap/heapam.c @@ -8481,7 +8481,6 @@ log_heap_new_cid(Relation relation, HeapTuple tup) { xlrec.cmin = InvalidCommandId; xlrec.cmax = HeapTupleHeaderGetRawCommandId(hdr); - } xlrec.combocid = InvalidCommandId; } diff --git a/src/backend/access/heap/vacuumlazy.c b/src/backend/access/heap/vacuumlazy.c index a1c2af33f1..788db569b2 100644 --- a/src/backend/access/heap/vacuumlazy.c +++ b/src/backend/access/heap/vacuumlazy.c @@ -2073,7 +2073,6 @@ lazy_scan_noprune(LVRelState *vacrel, elog(ERROR, "unexpected HeapTupleSatisfiesVacuum result"); break; } - } vacrel->offnum = InvalidOffsetNumber; diff --git a/src/backend/access/nbtree/nbtsplitloc.c b/src/backend/access/nbtree/nbtsplitloc.c index ee01ceafda..241e26d338 100644 --- a/src/backend/access/nbtree/nbtsplitloc.c +++ b/src/backend/access/nbtree/nbtsplitloc.c @@ -35,7 +35,6 @@ typedef struct /* split point identifying fields (returned by _bt_findsplitloc) */ OffsetNumber firstrightoff; /* first origpage item on rightpage */ bool newitemonleft; /* new item goes on left, or right? */ - } SplitPoint; typedef struct diff --git a/src/backend/access/nbtree/nbtutils.c b/src/backend/access/nbtree/nbtutils.c index fd1b53885c..ff260c393a 100644 --- a/src/backend/access/nbtree/nbtutils.c +++ b/src/backend/access/nbtree/nbtutils.c @@ -2116,14 +2116,12 @@ btoptions(Datum reloptions, bool validate) offsetof(BTOptions, vacuum_cleanup_index_scale_factor)}, {"deduplicate_items", RELOPT_TYPE_BOOL, offsetof(BTOptions, deduplicate_items)} - }; return (bytea *) build_reloptions(reloptions, validate, RELOPT_KIND_BTREE, sizeof(BTOptions), tab, lengthof(tab)); - } /* @@ -2591,7 +2589,6 @@ _bt_check_natts(Relation rel, bool heapkeyspace, Page page, OffsetNumber offnum) /* Use generic heapkeyspace pivot tuple handling */ } - } /* Handle heapkeyspace pivot tuples (excluding minus infinity items) */ diff --git a/src/backend/access/rmgrdesc/gistdesc.c b/src/backend/access/rmgrdesc/gistdesc.c index 9cab4fa158..d0c8e247c1 100644 --- a/src/backend/access/rmgrdesc/gistdesc.c +++ b/src/backend/access/rmgrdesc/gistdesc.c @@ -38,7 +38,6 @@ out_gistxlogDelete(StringInfo buf, gistxlogDelete *xlrec) { appendStringInfo(buf, "delete: latestRemovedXid %u, nitems: %u", xlrec->latestRemovedXid, xlrec->ntodelete); - } static void diff --git a/src/backend/access/spgist/spgutils.c b/src/backend/access/spgist/spgutils.c index 1ae7492216..a171ca8a08 100644 --- a/src/backend/access/spgist/spgutils.c +++ b/src/backend/access/spgist/spgutils.c @@ -747,7 +747,6 @@ spgoptions(Datum reloptions, bool validate) RELOPT_KIND_SPGIST, sizeof(SpGistOptions), tab, lengthof(tab)); - } /* diff --git a/src/backend/access/table/tableam.c b/src/backend/access/table/tableam.c index 4f20c6ac12..b3d1a6c3f8 100644 --- a/src/backend/access/table/tableam.c +++ b/src/backend/access/table/tableam.c @@ -381,7 +381,6 @@ simple_table_tuple_update(Relation rel, ItemPointer otid, elog(ERROR, "unrecognized table_tuple_update status: %u", result); break; } - } diff --git a/src/backend/access/transam/xlogrecovery.c b/src/backend/access/transam/xlogrecovery.c index 26be94b3f1..39ef865ed9 100644 --- a/src/backend/access/transam/xlogrecovery.c +++ b/src/backend/access/transam/xlogrecovery.c @@ -1790,7 +1790,6 @@ PerformWalRecovery(void) /* there are no WAL records following the checkpoint */ ereport(LOG, (errmsg("redo is not required"))); - } /* diff --git a/src/backend/catalog/objectaddress.c b/src/backend/catalog/objectaddress.c index ac6043514f..8377b4f7d4 100644 --- a/src/backend/catalog/objectaddress.c +++ b/src/backend/catalog/objectaddress.c @@ -1007,7 +1007,6 @@ get_object_address(ObjectType objtype, Node *object, address.objectId = get_domain_constraint_oid(domaddr.objectId, constrname, missing_ok); address.objectSubId = 0; - } break; case OBJECT_DATABASE: @@ -5621,7 +5620,6 @@ getObjectIdentityParts(const ObjectAddress *object, systable_endscan(rcscan); table_close(defaclrel, AccessShareLock); break; - } defacl = (Form_pg_default_acl) GETSTRUCT(tup); diff --git a/src/backend/catalog/pg_attrdef.c b/src/backend/catalog/pg_attrdef.c index 2d63c6e62a..c5d4a9912e 100644 --- a/src/backend/catalog/pg_attrdef.c +++ b/src/backend/catalog/pg_attrdef.c @@ -167,7 +167,6 @@ StoreAttrDefault(Relation rel, AttrNumber attnum, if (!missingIsNull) pfree(DatumGetPointer(missingval)); - } table_close(attrrel, RowExclusiveLock); heap_freetuple(atttup); diff --git a/src/backend/commands/copyto.c b/src/backend/commands/copyto.c index 55c38b04c4..643bbf286e 100644 --- a/src/backend/commands/copyto.c +++ b/src/backend/commands/copyto.c @@ -97,7 +97,6 @@ typedef struct CopyToStateData FmgrInfo *out_functions; /* lookup info for output functions */ MemoryContext rowcontext; /* per-row evaluation context */ uint64 bytes_processed; /* number of bytes processed so far */ - } CopyToStateData; /* DestReceiver for COPY (query) TO */ diff --git a/src/backend/commands/subscriptioncmds.c b/src/backend/commands/subscriptioncmds.c index 2e8d8afead..b94236f74d 100644 --- a/src/backend/commands/subscriptioncmds.c +++ b/src/backend/commands/subscriptioncmds.c @@ -1570,7 +1570,6 @@ DropSubscription(DropSubscriptionStmt *stmt, bool isTopLevel) */ if (slotname) ReplicationSlotDropAtPubNode(wrconn, slotname, false); - } PG_FINALLY(); { diff --git a/src/backend/commands/tablecmds.c b/src/backend/commands/tablecmds.c index 1d7db41d17..2cd8546d47 100644 --- a/src/backend/commands/tablecmds.c +++ b/src/backend/commands/tablecmds.c @@ -945,7 +945,6 @@ DefineRelation(CreateStmt *stmt, char relkind, Oid ownerId, ereport(ERROR, (errcode(ERRCODE_FEATURE_NOT_SUPPORTED), errmsg("specifying a table access method is not supported on a partitioned table"))); - } else if (RELKIND_HAS_TABLE_AM(relkind)) accessMethod = default_table_access_method; @@ -3773,7 +3772,6 @@ RenameConstraint(RenameStmt *stmt) stmt->relation->inh), /* recursive? */ false, /* recursing? */ 0 /* expected inhcount */ ); - } /* @@ -17451,7 +17449,6 @@ ComputePartitionAttrs(ParseState *pstate, Relation rel, List *partParams, AttrNu errmsg("data type %s has no default operator class for access method \"%s\"", format_type_be(atttype), "btree"), errhint("You must specify a btree operator class or define a default btree operator class for the data type."))); - } } else diff --git a/src/backend/executor/execParallel.c b/src/backend/executor/execParallel.c index 9a0d5d59ef..f1fd7f7e8b 100644 --- a/src/backend/executor/execParallel.c +++ b/src/backend/executor/execParallel.c @@ -1439,7 +1439,6 @@ ParallelQueryMain(dsm_segment *seg, shm_toc *toc) paramexec_space = dsa_get_address(area, fpes->param_exec); RestoreParamExecParams(paramexec_space, queryDesc->estate); - } pwcxt.toc = toc; pwcxt.seg = seg; diff --git a/src/backend/executor/execReplication.c b/src/backend/executor/execReplication.c index 27989bd723..b000645d48 100644 --- a/src/backend/executor/execReplication.c +++ b/src/backend/executor/execReplication.c @@ -548,7 +548,6 @@ ExecSimpleRelationDelete(ResultRelInfo *resultRelInfo, { skip_tuple = !ExecBRDeleteTriggers(estate, epqstate, resultRelInfo, tid, NULL, NULL); - } if (!skip_tuple) diff --git a/src/backend/executor/nodeAgg.c b/src/backend/executor/nodeAgg.c index 23030a32a5..3223d9b24e 100644 --- a/src/backend/executor/nodeAgg.c +++ b/src/backend/executor/nodeAgg.c @@ -4119,7 +4119,6 @@ build_pertrans_for_aggref(AggStatePerTrans pertrans, 2, InvalidOid, (void *) aggstate, NULL); - } /* diff --git a/src/backend/executor/nodeMergejoin.c b/src/backend/executor/nodeMergejoin.c index edb8972c5b..864e3baf86 100644 --- a/src/backend/executor/nodeMergejoin.c +++ b/src/backend/executor/nodeMergejoin.c @@ -1674,5 +1674,4 @@ ExecReScanMergeJoin(MergeJoinState *node) ExecReScan(node->js.ps.lefttree); if (node->js.ps.righttree->chgParam == NULL) ExecReScan(node->js.ps.righttree); - } diff --git a/src/backend/jit/llvm/llvmjit_deform.c b/src/backend/jit/llvm/llvmjit_deform.c index a3355860c3..661f15272b 100644 --- a/src/backend/jit/llvm/llvmjit_deform.c +++ b/src/backend/jit/llvm/llvmjit_deform.c @@ -192,7 +192,6 @@ slot_compile_deform(LLVMJitContext *context, TupleDesc desc, v_tupleheaderp = l_load_struct_gep(b, v_heapslot, FIELDNO_HEAPTUPLETABLESLOT_TUPLE, "tupleheader"); - } else if (ops == &TTSOpsMinimalTuple) { @@ -357,7 +356,6 @@ slot_compile_deform(LLVMJitContext *context, TupleDesc desc, LLVMAddCase(v_switch, v_attno, attcheckattnoblocks[attnum]); } - } else { diff --git a/src/backend/jit/llvm/llvmjit_expr.c b/src/backend/jit/llvm/llvmjit_expr.c index 9c8f341d96..b6b6512ef1 100644 --- a/src/backend/jit/llvm/llvmjit_expr.c +++ b/src/backend/jit/llvm/llvmjit_expr.c @@ -2173,7 +2173,6 @@ llvm_compile_expr(ExprState *state) ""); LLVMBuildBr(b, opblocks[opno + 1]); - } LLVMPositionBuilderAtEnd(b, b_no_init); diff --git a/src/backend/libpq/hba.c b/src/backend/libpq/hba.c index cbd17b790d..327a4b42af 100644 --- a/src/backend/libpq/hba.c +++ b/src/backend/libpq/hba.c @@ -1775,7 +1775,6 @@ parse_hba_auth_opt(char *name, char *val, HbaLine *hbaline, hbaline->pam_use_hostname = true; else hbaline->pam_use_hostname = false; - } else if (strcmp(name, "ldapurl") == 0) { diff --git a/src/backend/optimizer/geqo/geqo_cx.c b/src/backend/optimizer/geqo/geqo_cx.c index 3b8d2fe560..34cc53af53 100644 --- a/src/backend/optimizer/geqo/geqo_cx.c +++ b/src/backend/optimizer/geqo/geqo_cx.c @@ -115,7 +115,6 @@ cx(PlannerInfo *root, Gene *tour1, Gene *tour2, Gene *offspring, for (i = 0; i < num_gene; i++) if (tour1[i] != offspring[i]) num_diffs++; - } return num_diffs; diff --git a/src/backend/optimizer/geqo/geqo_erx.c b/src/backend/optimizer/geqo/geqo_erx.c index 3b92f420e0..cc0661365f 100644 --- a/src/backend/optimizer/geqo/geqo_erx.c +++ b/src/backend/optimizer/geqo/geqo_erx.c @@ -222,11 +222,9 @@ gimme_tour(PlannerInfo *root, Edge *edge_table, Gene *new_gene, int num_gene) /* mark this node as incorporated */ edge_table[(int) new_gene[i - 1]].unused_edges = -1; - } /* for (i=1; idefname, action))); - } else if (strcmp(defel->defname, "reserve_wal") == 0) { diff --git a/src/backend/statistics/dependencies.c b/src/backend/statistics/dependencies.c index 34326d5561..b6f3184961 100644 --- a/src/backend/statistics/dependencies.c +++ b/src/backend/statistics/dependencies.c @@ -199,7 +199,6 @@ DependencyGenerator_free(DependencyGenerator state) { pfree(state->dependencies); pfree(state); - } /* generate next combination */ diff --git a/src/backend/storage/buffer/bufmgr.c b/src/backend/storage/buffer/bufmgr.c index c12028ca0f..e02ea3a977 100644 --- a/src/backend/storage/buffer/bufmgr.c +++ b/src/backend/storage/buffer/bufmgr.c @@ -2676,7 +2676,6 @@ CheckForBufferLeaks(void) PrintBufferLeakWarning(res->buffer); RefCountErrors++; } - } Assert(RefCountErrors == 0); @@ -3652,7 +3651,6 @@ FlushRelationsAllBuffers(SMgrRelation *smgrs, int nrels) break; } } - } else { diff --git a/src/backend/storage/buffer/freelist.c b/src/backend/storage/buffer/freelist.c index 3b98e68d50..990e081aae 100644 --- a/src/backend/storage/buffer/freelist.c +++ b/src/backend/storage/buffer/freelist.c @@ -308,7 +308,6 @@ StrategyGetBuffer(BufferAccessStrategy strategy, uint32 *buf_state) return buf; } UnlockBufHdr(buf, local_buf_state); - } } diff --git a/src/backend/storage/ipc/latch.c b/src/backend/storage/ipc/latch.c index c3aaa8bff0..9dbb4f17cf 100644 --- a/src/backend/storage/ipc/latch.c +++ b/src/backend/storage/ipc/latch.c @@ -648,7 +648,6 @@ SetLatch(Latch *latch) */ } #endif - } /* diff --git a/src/backend/storage/ipc/procarray.c b/src/backend/storage/ipc/procarray.c index cb39fdde33..f6e98aae29 100644 --- a/src/backend/storage/ipc/procarray.c +++ b/src/backend/storage/ipc/procarray.c @@ -243,7 +243,6 @@ typedef struct ComputeXidHorizonsResult * session's temporary tables. */ TransactionId temp_oldest_nonremovable; - } ComputeXidHorizonsResult; /* @@ -1839,7 +1838,6 @@ ComputeXidHorizons(ComputeXidHorizonsResult *h) /* Catalog tables need to consider all backends in this db */ h->catalog_oldest_nonremovable = TransactionIdOlder(h->catalog_oldest_nonremovable, xmin); - } } diff --git a/src/backend/storage/lmgr/lwlock.c b/src/backend/storage/lmgr/lwlock.c index c24779d0bb..fef462b110 100644 --- a/src/backend/storage/lmgr/lwlock.c +++ b/src/backend/storage/lmgr/lwlock.c @@ -1096,7 +1096,6 @@ LWLockQueueSelf(LWLock *lock, LWLockMode mode) #ifdef LOCK_DEBUG pg_atomic_fetch_add_u32(&lock->nwaiters, 1); #endif - } /* diff --git a/src/backend/storage/page/bufpage.c b/src/backend/storage/page/bufpage.c index 366d57ea7a..cc15396789 100644 --- a/src/backend/storage/page/bufpage.c +++ b/src/backend/storage/page/bufpage.c @@ -563,7 +563,6 @@ compactify_tuples(itemIdCompact itemidbase, int nitems, Page page, bool presorte /* update the line pointer to reference the new offset */ lp->lp_off = upper; - } /* move the remaining tuples. */ @@ -669,7 +668,6 @@ compactify_tuples(itemIdCompact itemidbase, int nitems, Page page, bool presorte /* update the line pointer to reference the new offset */ lp->lp_off = upper; - } /* Copy the remaining chunk */ diff --git a/src/backend/storage/sync/sync.c b/src/backend/storage/sync/sync.c index c695d816fc..e1fb631003 100644 --- a/src/backend/storage/sync/sync.c +++ b/src/backend/storage/sync/sync.c @@ -162,7 +162,6 @@ InitSync(void) HASH_ELEM | HASH_BLOBS | HASH_CONTEXT); pendingUnlinks = NIL; } - } /* diff --git a/src/backend/tcop/postgres.c b/src/backend/tcop/postgres.c index 95dc2e2c83..304cce135a 100644 --- a/src/backend/tcop/postgres.c +++ b/src/backend/tcop/postgres.c @@ -2636,7 +2636,6 @@ exec_describe_statement_message(const char *stmt_name) } else pq_putemptymessage('n'); /* NoData */ - } /* diff --git a/src/backend/tcop/utility.c b/src/backend/tcop/utility.c index f364a9b88a..0e7b7b3138 100644 --- a/src/backend/tcop/utility.c +++ b/src/backend/tcop/utility.c @@ -3741,7 +3741,6 @@ GetCommandLogLevel(Node *parsetree) lev = LOGSTMT_ALL; break; } - } break; diff --git a/src/backend/tsearch/to_tsany.c b/src/backend/tsearch/to_tsany.c index 1b0a1e244b..edeffacc2d 100644 --- a/src/backend/tsearch/to_tsany.c +++ b/src/backend/tsearch/to_tsany.c @@ -569,7 +569,6 @@ pushval_morph(Datum opaque, TSQueryParserState state, char *strval, int lenval, } pfree(prs.words); - } else pushStop(state); @@ -720,5 +719,4 @@ websearch_to_tsquery(PG_FUNCTION_ARGS) PG_RETURN_DATUM(DirectFunctionCall2(websearch_to_tsquery_byid, ObjectIdGetDatum(cfgId), PointerGetDatum(in))); - } diff --git a/src/backend/tsearch/ts_parse.c b/src/backend/tsearch/ts_parse.c index fe46988161..27b2cca2df 100644 --- a/src/backend/tsearch/ts_parse.c +++ b/src/backend/tsearch/ts_parse.c @@ -584,7 +584,6 @@ hlparsetext(Oid cfgId, HeadlineParsedText *prs, TSQuery query, char *buf, int bu else addHLParsedLex(prs, query, lexs, NULL); } while (norms); - } while (type > 0); FunctionCall1(&(prsobj->prsend), PointerGetDatum(prsdata)); @@ -629,7 +628,6 @@ generateHeadline(HeadlineParsedText *prs) memcpy(ptr, prs->fragdelim, prs->fragdelimlen); ptr += prs->fragdelimlen; } - } if (wrd->replace) { diff --git a/src/backend/utils/activity/pgstat_database.c b/src/backend/utils/activity/pgstat_database.c index 649d9c6960..d9275611f0 100644 --- a/src/backend/utils/activity/pgstat_database.c +++ b/src/backend/utils/activity/pgstat_database.c @@ -328,7 +328,6 @@ pgstat_prep_database_pending(Oid dboid) NULL); return entry_ref->pending; - } /* diff --git a/src/backend/utils/adt/acl.c b/src/backend/utils/adt/acl.c index 8bdba1c42a..772c04155c 100644 --- a/src/backend/utils/adt/acl.c +++ b/src/backend/utils/adt/acl.c @@ -3094,7 +3094,6 @@ convert_database_priv_string(text *priv_type_text) }; return convert_any_priv_string(priv_type_text, database_priv_map); - } diff --git a/src/backend/utils/adt/json.c b/src/backend/utils/adt/json.c index da3f1b9700..63649ba735 100644 --- a/src/backend/utils/adt/json.c +++ b/src/backend/utils/adt/json.c @@ -1489,7 +1489,6 @@ json_object(PG_FUNCTION_ARGS) pfree(result.data); PG_RETURN_TEXT_P(rval); - } /* diff --git a/src/backend/utils/adt/jsonb.c b/src/backend/utils/adt/jsonb.c index 2043f2e74a..56c588bbab 100644 --- a/src/backend/utils/adt/jsonb.c +++ b/src/backend/utils/adt/jsonb.c @@ -851,7 +851,6 @@ datum_to_jsonb(Datum val, bool is_null, JsonbInState *result, sem.object_field_start = jsonb_in_object_field_start; pg_parse_json_or_ereport(lex, &sem); - } break; case JSONBTYPE_JSONB: diff --git a/src/backend/utils/adt/jsonfuncs.c b/src/backend/utils/adt/jsonfuncs.c index a682d9c973..d1356d6416 100644 --- a/src/backend/utils/adt/jsonfuncs.c +++ b/src/backend/utils/adt/jsonfuncs.c @@ -4168,7 +4168,6 @@ json_strip_nulls(PG_FUNCTION_ARGS) PG_RETURN_TEXT_P(cstring_to_text_with_len(state->strval->data, state->strval->len)); - } /* diff --git a/src/backend/utils/adt/multirangetypes.c b/src/backend/utils/adt/multirangetypes.c index efd8584a3d..67d7d67fb8 100644 --- a/src/backend/utils/adt/multirangetypes.c +++ b/src/backend/utils/adt/multirangetypes.c @@ -1181,7 +1181,6 @@ multirange_minus_internal(Oid mltrngtypoid, TypeCacheEntry *rangetyp, */ range_count3++; r2 = ++i2 >= range_count2 ? NULL : ranges2[i2]; - } else if (range_overlaps_internal(rangetyp, r1, r2)) { @@ -1200,7 +1199,6 @@ multirange_minus_internal(Oid mltrngtypoid, TypeCacheEntry *rangetyp, break; else r2 = ++i2 >= range_count2 ? NULL : ranges2[i2]; - } else { diff --git a/src/backend/utils/adt/oracle_compat.c b/src/backend/utils/adt/oracle_compat.c index f1673cc145..a6e043c32c 100644 --- a/src/backend/utils/adt/oracle_compat.c +++ b/src/backend/utils/adt/oracle_compat.c @@ -868,7 +868,6 @@ translate(PG_FUNCTION_ARGS) target += len; retlen += len; } - } else { diff --git a/src/backend/utils/adt/ruleutils.c b/src/backend/utils/adt/ruleutils.c index 3296ad070e..7e08d7fe6c 100644 --- a/src/backend/utils/adt/ruleutils.c +++ b/src/backend/utils/adt/ruleutils.c @@ -9440,7 +9440,6 @@ get_rule_expr(Node *node, deparse_context *context, get_rule_expr_paren((Node *) xexpr->args, context, false, node); break; } - } if (xexpr->op == IS_XMLSERIALIZE) appendStringInfo(buf, " AS %s", diff --git a/src/backend/utils/adt/selfuncs.c b/src/backend/utils/adt/selfuncs.c index fb4fb987e7..71cbc1c3d8 100644 --- a/src/backend/utils/adt/selfuncs.c +++ b/src/backend/utils/adt/selfuncs.c @@ -3628,7 +3628,6 @@ estimate_num_groups_incremental(PlannerInfo *root, List *groupExprs, */ if (estinfo != NULL && varinfo2->isdefault) estinfo->flags |= SELFLAG_USED_DEFAULT; - } /* we're done with this relation */ diff --git a/src/backend/utils/adt/timestamp.c b/src/backend/utils/adt/timestamp.c index 70937eaa46..da73796eac 100644 --- a/src/backend/utils/adt/timestamp.c +++ b/src/backend/utils/adt/timestamp.c @@ -4898,7 +4898,6 @@ timestamp_part_common(PG_FUNCTION_ARGS, bool retnumeric) lowunits, format_type_be(TIMESTAMPOID)))); intresult = 0; } - } else { @@ -5123,7 +5122,6 @@ timestamptz_part_common(PG_FUNCTION_ARGS, bool retnumeric) lowunits, format_type_be(TIMESTAMPTZOID)))); intresult = 0; } - } else if (type == RESERV) { diff --git a/src/backend/utils/adt/tsvector_op.c b/src/backend/utils/adt/tsvector_op.c index 8ba9ef22f5..addc349151 100644 --- a/src/backend/utils/adt/tsvector_op.c +++ b/src/backend/utils/adt/tsvector_op.c @@ -2196,7 +2196,6 @@ insertStatEntry(MemoryContext persistentContext, TSVectorStat *stat, TSVector tx else pnode->right = node; } - } else { diff --git a/src/backend/utils/cache/catcache.c b/src/backend/utils/cache/catcache.c index ec073e1ed0..6ae7c1f50b 100644 --- a/src/backend/utils/cache/catcache.c +++ b/src/backend/utils/cache/catcache.c @@ -1710,7 +1710,6 @@ SearchCatCacheList(CatCache *cache, * we'd better do so before we start marking the members as belonging * to the list. */ - } PG_CATCH(); { @@ -1962,7 +1961,6 @@ CatCacheCopyKeys(TupleDesc tupdesc, int nkeys, int *attnos, att->attbyval, att->attlen); } - } /* diff --git a/src/backend/utils/error/elog.c b/src/backend/utils/error/elog.c index 72778b896a..55ee5423af 100644 --- a/src/backend/utils/error/elog.c +++ b/src/backend/utils/error/elog.c @@ -2683,7 +2683,6 @@ log_line_prefix(StringInfo buf, ErrorData *edata) appendStringInfo(buf, "%*s", padding, psdisp); else appendBinaryStringInfo(buf, psdisp, displen); - } else if (padding != 0) appendStringInfoSpaces(buf, @@ -2722,7 +2721,6 @@ log_line_prefix(StringInfo buf, ErrorData *edata) appendStringInfo(buf, "(%s)", MyProcPort->remote_port); } - } else if (padding != 0) appendStringInfoSpaces(buf, diff --git a/src/backend/utils/misc/guc.c b/src/backend/utils/misc/guc.c index 9e0f262088..8e9b71375c 100644 --- a/src/backend/utils/misc/guc.c +++ b/src/backend/utils/misc/guc.c @@ -5906,7 +5906,6 @@ InitializeWalConsistencyChecking(void) /* checking should not be deferred again */ Assert(!check_wal_consistency_checking_deferred); } - } /* diff --git a/src/backend/utils/mmgr/generation.c b/src/backend/utils/mmgr/generation.c index 56ed496386..e530e272e0 100644 --- a/src/backend/utils/mmgr/generation.c +++ b/src/backend/utils/mmgr/generation.c @@ -620,7 +620,6 @@ GenerationBlockMarkEmpty(GenerationBlock *block) block->nchunks = 0; block->nfree = 0; block->freeptr = ((char *) block) + Generation_BLOCKHDRSZ; - } /* diff --git a/src/backend/utils/resowner/resowner.c b/src/backend/utils/resowner/resowner.c index 3236b1b919..ceb4b0e3f7 100644 --- a/src/backend/utils/resowner/resowner.c +++ b/src/backend/utils/resowner/resowner.c @@ -896,7 +896,6 @@ CreateAuxProcessResourceOwner(void) * owner. (This needs to run after, e.g., ShutdownXLOG.) */ on_shmem_exit(ReleaseAuxProcessResourcesCallback, 0); - } /* diff --git a/src/backend/utils/sort/tuplesort.c b/src/backend/utils/sort/tuplesort.c index 571fb95532..1174e1a31c 100644 --- a/src/backend/utils/sort/tuplesort.c +++ b/src/backend/utils/sort/tuplesort.c @@ -3186,7 +3186,6 @@ mergeonerun(Tuplesortstate *state) { stup.srctape = srcTapeIndex; tuplesort_heap_replace_top(state, &stup); - } else { diff --git a/src/bin/initdb/initdb.c b/src/bin/initdb/initdb.c index ab826da650..1cb4a5b0d2 100644 --- a/src/bin/initdb/initdb.c +++ b/src/bin/initdb/initdb.c @@ -2407,7 +2407,6 @@ setup_locale_encoding(void) if (!check_locale_encoding(lc_ctype, encodingid) || !check_locale_encoding(lc_collate, encodingid)) exit(1); /* check_locale_encoding printed the error */ - } @@ -2486,7 +2485,6 @@ setup_text_search(void) printf(_("The default text search configuration will be set to \"%s\".\n"), default_text_search_config); - } diff --git a/src/bin/pg_dump/pg_dumpall.c b/src/bin/pg_dump/pg_dumpall.c index 6ef3d61421..79a723885e 100644 --- a/src/bin/pg_dump/pg_dumpall.c +++ b/src/bin/pg_dump/pg_dumpall.c @@ -1373,7 +1373,6 @@ dumpDatabases(PGconn *conn) pg_fatal("could not re-open the output file \"%s\": %m", filename); } - } PQclear(res); diff --git a/src/bin/pg_rewind/parsexlog.c b/src/bin/pg_rewind/parsexlog.c index 87b9f75f2c..c6792dafae 100644 --- a/src/bin/pg_rewind/parsexlog.c +++ b/src/bin/pg_rewind/parsexlog.c @@ -98,7 +98,6 @@ extractPageMap(const char *datadir, XLogRecPtr startpoint, int tliIndex, } extractPageInfo(xlogreader); - } while (xlogreader->EndRecPtr < endpoint); /* diff --git a/src/bin/pg_verifybackup/pg_verifybackup.c b/src/bin/pg_verifybackup/pg_verifybackup.c index 6671a84150..bd18b4491d 100644 --- a/src/bin/pg_verifybackup/pg_verifybackup.c +++ b/src/bin/pg_verifybackup/pg_verifybackup.c @@ -740,8 +740,6 @@ verify_file_checksum(verifier_context *context, manifest_file *m, close(fd); return; } - - } if (rc < 0) report_backup_error(context, "could not read file \"%s\": %m", diff --git a/src/bin/psql/describe.c b/src/bin/psql/describe.c index e7377d4583..1a2c6bc7f5 100644 --- a/src/bin/psql/describe.c +++ b/src/bin/psql/describe.c @@ -2622,7 +2622,6 @@ describeOneTableDetails(const char *schemaname, PQgetvalue(result, i, 4)); printTableAddFooter(&cont, buf.data); - } PQclear(result); } @@ -3172,7 +3171,6 @@ describeOneTableDetails(const char *schemaname, case 4: printfPQExpBuffer(&buf, _("Triggers firing on replica only:")); break; - } printTableAddFooter(&cont, buf.data); have_heading = true; diff --git a/src/bin/psql/prompt.c b/src/bin/psql/prompt.c index 10ab390531..509e6422b7 100644 --- a/src/bin/psql/prompt.c +++ b/src/bin/psql/prompt.c @@ -316,7 +316,6 @@ get_prompt(promptStatus_t status, ConditionalStack cstack) buf[0] = *p; buf[1] = '\0'; break; - } esc = false; } diff --git a/src/common/jsonapi.c b/src/common/jsonapi.c index 6666077a93..98e4ef0942 100644 --- a/src/common/jsonapi.c +++ b/src/common/jsonapi.c @@ -658,7 +658,6 @@ json_lex(JsonLexContext *lex) lex->token_type = JSON_TOKEN_FALSE; else return JSON_INVALID_TOKEN; - } } /* end of switch */ } @@ -856,7 +855,6 @@ json_lex_string(JsonLexContext *lex) lex->token_terminator = s + pg_encoding_mblen_bounded(lex->input_encoding, s); return JSON_ESCAPING_INVALID; } - } else if (lex->strval != NULL) { @@ -865,7 +863,6 @@ json_lex_string(JsonLexContext *lex) appendStringInfoChar(lex->strval, *s); } - } if (hi_surrogate != -1) diff --git a/src/fe_utils/print.c b/src/fe_utils/print.c index dcdb2e0d0c..fe676a971b 100644 --- a/src/fe_utils/print.c +++ b/src/fe_utils/print.c @@ -1173,7 +1173,6 @@ print_aligned_text(const printTableContent *cont, FILE *fout, bool is_pager) if (opt_border == 2) fputs(dformat->rightvrule, fout); fputc('\n', fout); - } while (more_lines); } diff --git a/src/interfaces/ecpg/ecpglib/connect.c b/src/interfaces/ecpg/ecpglib/connect.c index a2166b1e12..8dfcabe3b5 100644 --- a/src/interfaces/ecpg/ecpglib/connect.c +++ b/src/interfaces/ecpg/ecpglib/connect.c @@ -315,7 +315,6 @@ ECPGconnect(int lineno, int c, const char *name, const char *user, const char *p ecpg_free(dbname); dbname = ecpg_strdup(envname, lineno); } - } if (dbname == NULL && connection_name == NULL) diff --git a/src/interfaces/ecpg/ecpglib/data.c b/src/interfaces/ecpg/ecpglib/data.c index 6bc91ef7eb..a1bba42bb8 100644 --- a/src/interfaces/ecpg/ecpglib/data.c +++ b/src/interfaces/ecpg/ecpglib/data.c @@ -564,7 +564,6 @@ ecpg_get_data(const PGresult *results, int act_tuple, int act_field, int lineno, } pval += size; - } break; diff --git a/src/interfaces/ecpg/ecpglib/execute.c b/src/interfaces/ecpg/ecpglib/execute.c index 930b6adbe4..6a7ef0bbf6 100644 --- a/src/interfaces/ecpg/ecpglib/execute.c +++ b/src/interfaces/ecpg/ecpglib/execute.c @@ -1392,7 +1392,6 @@ ecpg_build_params(struct statement *stmt) if (sqlda->sqln == desc_counter) desc_counter = 0; } - } else { diff --git a/src/interfaces/ecpg/ecpglib/misc.c b/src/interfaces/ecpg/ecpglib/misc.c index b8dbe5e7f3..1eef1ec044 100644 --- a/src/interfaces/ecpg/ecpglib/misc.c +++ b/src/interfaces/ecpg/ecpglib/misc.c @@ -191,7 +191,6 @@ ECPGtransactionStatus(const char *connection_name) } return PQtransactionStatus(con->connection); - } bool diff --git a/src/interfaces/ecpg/pgtypeslib/dt_common.c b/src/interfaces/ecpg/pgtypeslib/dt_common.c index e8a8a0f0ed..e0fae3d5f1 100644 --- a/src/interfaces/ecpg/pgtypeslib/dt_common.c +++ b/src/interfaces/ecpg/pgtypeslib/dt_common.c @@ -1740,7 +1740,6 @@ ParseDateTime(char *timestr, char *lowstr, { (*endstr)++; continue; - } /* otherwise, something is not right... */ else diff --git a/src/interfaces/ecpg/pgtypeslib/interval.c b/src/interfaces/ecpg/pgtypeslib/interval.c index a7e530cb5d..73bde94aaf 100644 --- a/src/interfaces/ecpg/pgtypeslib/interval.c +++ b/src/interfaces/ecpg/pgtypeslib/interval.c @@ -947,7 +947,6 @@ interval2tm(interval span, struct tm *tm, fsec_t *fsec) { tm->tm_year = span.month / MONTHS_PER_YEAR; tm->tm_mon = span.month % MONTHS_PER_YEAR; - } else { diff --git a/src/interfaces/ecpg/pgtypeslib/numeric.c b/src/interfaces/ecpg/pgtypeslib/numeric.c index dbe4eb21e2..34efc9045f 100644 --- a/src/interfaces/ecpg/pgtypeslib/numeric.c +++ b/src/interfaces/ecpg/pgtypeslib/numeric.c @@ -1308,7 +1308,6 @@ PGTYPESnumeric_cmp(numeric *var1, numeric *var2) errno = PGTYPES_NUM_BAD_NUMERIC; return INT_MAX; - } int diff --git a/src/interfaces/ecpg/pgtypeslib/timestamp.c b/src/interfaces/ecpg/pgtypeslib/timestamp.c index 3f82ee54a5..54cc04addd 100644 --- a/src/interfaces/ecpg/pgtypeslib/timestamp.c +++ b/src/interfaces/ecpg/pgtypeslib/timestamp.c @@ -863,8 +863,6 @@ PGTYPEStimestamp_add_interval(timestamp * tin, interval * span, timestamp * tout { if (TIMESTAMP_NOT_FINITE(*tin)) *tout = *tin; - - else { if (span->month != 0) @@ -873,7 +871,6 @@ PGTYPEStimestamp_add_interval(timestamp * tin, interval * span, timestamp * tout *tm = &tt; fsec_t fsec; - if (timestamp2tm(*tin, NULL, tm, &fsec, NULL) != 0) return -1; tm->tm_mon += span->month; @@ -898,12 +895,11 @@ PGTYPEStimestamp_add_interval(timestamp * tin, interval * span, timestamp * tout return -1; } - *tin += span->time; *tout = *tin; } - return 0; + return 0; } @@ -925,6 +921,5 @@ PGTYPEStimestamp_sub_interval(timestamp * tin, interval * span, timestamp * tout tspan.month = -span->month; tspan.time = -span->time; - return PGTYPEStimestamp_add_interval(tin, &tspan, tout); } diff --git a/src/interfaces/libpq/fe-connect.c b/src/interfaces/libpq/fe-connect.c index cf554d389f..4c12f1411f 100644 --- a/src/interfaces/libpq/fe-connect.c +++ b/src/interfaces/libpq/fe-connect.c @@ -664,7 +664,6 @@ PQconnectdbParams(const char *const *keywords, (void) connectDBComplete(conn); return conn; - } /* diff --git a/src/interfaces/libpq/fe-secure-openssl.c b/src/interfaces/libpq/fe-secure-openssl.c index 24a598b6e4..42d8d4616e 100644 --- a/src/interfaces/libpq/fe-secure-openssl.c +++ b/src/interfaces/libpq/fe-secure-openssl.c @@ -1432,7 +1432,6 @@ initialize_SSL(PGconn *conn) } SSLerrfree(err); - } } diff --git a/src/pl/tcl/pltcl.c b/src/pl/tcl/pltcl.c index 68c9bd1970..0dd6d8ab2c 100644 --- a/src/pl/tcl/pltcl.c +++ b/src/pl/tcl/pltcl.c @@ -1243,7 +1243,6 @@ pltcl_trigger_handler(PG_FUNCTION_ARGS, pltcl_call_state *call_state, for (i = 0; i < trigdata->tg_trigger->tgnargs; i++) Tcl_ListObjAppendElement(NULL, tcl_cmd, Tcl_NewStringObj(utf_e2u(trigdata->tg_trigger->tgargs[i]), -1)); - } PG_CATCH(); { diff --git a/src/port/chklocale.c b/src/port/chklocale.c index 3e77725352..c85d8da3c8 100644 --- a/src/port/chklocale.c +++ b/src/port/chklocale.c @@ -278,7 +278,6 @@ win32_langinfo(const char *ctype) strcpy(r, codepage); } } - } #endif diff --git a/src/test/modules/ssl_passphrase_callback/ssl_passphrase_func.c b/src/test/modules/ssl_passphrase_callback/ssl_passphrase_func.c index 7c469fd57e..b5bb5580a0 100644 --- a/src/test/modules/ssl_passphrase_callback/ssl_passphrase_func.c +++ b/src/test/modules/ssl_passphrase_callback/ssl_passphrase_func.c @@ -89,5 +89,4 @@ rot13_passphrase(char *buf, int size, int rwflag, void *userdata) } return strlen(buf); - }