Remove unnecessary semicolons after goto labels

According to the C standard, a label must followed by a statement.
If there was ever a time we needed an empty statement here, it was
a long time ago.

Japin Li

Reviewed by Julien Rouhaud
Discussion: https://www.postgresql.org/message-id/MEYP282MB16690F40189A4F060B41D56DB65E9%40MEYP282MB1669.AUSP282.PROD.OUTLOOK.COM

diff --git a/src/backend/access/transam/slru.c b/src/backend/access/transam/slru.c
index b65cb49d7ff235c02c3fa683fa22d6e545cf41b9..6c57fae31216397fb0cfa16b51b5e8c24347983f 100644 (file)
--- a/src/backend/access/transam/slru.c
+++ b/src/backend/access/transam/slru.c
@@ -1239,7 +1239,7 @@ SimpleLruTruncate(SlruCtl ctl, int cutoffPage)
         */
        LWLockAcquire(shared->ControlLock, LW_EXCLUSIVE);
 
-restart:;
+restart:
 
        /*
         * While we are holding the lock, make an important safety check: the

diff --git a/src/backend/executor/nodeModifyTable.c b/src/backend/executor/nodeModifyTable.c
index 04454ad6e60be3dfd825e5c3b6acdf7f04188810..447f7bc2fb7883b8a63c9ee8df9cb10f2a3ef544 100644 (file)
--- a/src/backend/executor/nodeModifyTable.c
+++ b/src/backend/executor/nodeModifyTable.c
@@ -1373,7 +1373,7 @@ ExecDelete(ModifyTableContext *context,
                 * special-case behavior needed for referential integrity updates in
                 * transaction-snapshot mode transactions.
                 */
-ldelete:;
+ldelete:
                result = ExecDeleteAct(context, resultRelInfo, tupleid, changingPart);
 
                switch (result)
@@ -1855,7 +1855,7 @@ ExecUpdateAct(ModifyTableContext *context, ResultRelInfo *resultRelInfo,
         * then trigger.c will have done table_tuple_lock to lock the correct
         * tuple, so there's no need to do them again.)
         */
-lreplace:;
+lreplace:
 
        /* ensure slot is independent, consider e.g. EPQ */
        ExecMaterializeSlot(slot);
@@ -2686,7 +2686,7 @@ ExecMergeMatched(ModifyTableContext *context, ResultRelInfo *resultRelInfo,
        econtext->ecxt_innertuple = context->planSlot;
        econtext->ecxt_outertuple = NULL;
 
-lmerge_matched:;
+lmerge_matched:
 
        /*
         * This routine is only invoked for matched rows, and we must have found