Remove unnecessary commas for goto labels - Mailing list pgsql-hackers

From Japin Li
Subject Remove unnecessary commas for goto labels
Date
Msg-id MEYP282MB16690F40189A4F060B41D56DB65E9@MEYP282MB1669.AUSP282.PROD.OUTLOOK.COM
Whole thread Raw
Responses Re: Remove unnecessary commas for goto labels  (Julien Rouhaud <rjuju123@gmail.com>)
List pgsql-hackers
Hi hackers,

I find there are some unnecessary commas for goto lables,
attached a patch to remove them.

-- 
Regrads,
Japin Li.
ChengDu WenWu Information Technology Co.,Ltd.

From 3ac7d03e1c5d81e883406ac62fafbe7e7d32fd1e Mon Sep 17 00:00:00 2001
From: Japin Li <japinli@hotmail.com>
Date: Sun, 9 Oct 2022 07:35:43 +0800
Subject: [PATCH v1 1/1] Remove unnecessary commas for goto labels

---
 src/backend/access/transam/slru.c      | 2 +-
 src/backend/executor/nodeModifyTable.c | 6 +++---
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/src/backend/access/transam/slru.c b/src/backend/access/transam/slru.c
index b65cb49d7f..6c57fae312 100644
--- 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 04454ad6e6..447f7bc2fb 100644
--- 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
-- 
2.17.1


pgsql-hackers by date:

Previous
From: Andrew Borodin
Date:
Subject: Re: Amcheck verification of GiST and GIN
Next
From: Michael Paquier
Date:
Subject: Re: use has_privs_of_role() for pg_hba.conf