Remove double trailing semicolons

Started by David Rowleyabout 7 years ago4 messages
#1David Rowley
david.rowley@2ndquadrant.com
1 attachment(s)

While looking over some recent commits, I noticed there are some code
lines with a double trailing semicolon instead of a single one. The
attached fixes these.

--
David Rowley http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training & Services

Attachments:

remove_double_trailing_semicolons.patchapplication/octet-stream; name=remove_double_trailing_semicolons.patchDownload
diff --git a/src/backend/commands/trigger.c b/src/backend/commands/trigger.c
index bcdd86ce92..fb0de60a45 100644
--- a/src/backend/commands/trigger.c
+++ b/src/backend/commands/trigger.c
@@ -5780,7 +5780,7 @@ AfterTriggerSaveEvent(EState *estate, ResultRelInfo *relinfo,
 		bool		delete_old_table = transition_capture->tcs_delete_old_table;
 		bool		update_old_table = transition_capture->tcs_update_old_table;
 		bool		update_new_table = transition_capture->tcs_update_new_table;
-		bool		insert_new_table = transition_capture->tcs_insert_new_table;;
+		bool		insert_new_table = transition_capture->tcs_insert_new_table;
 
 		/*
 		 * For INSERT events newtup should be non-NULL, for DELETE events
diff --git a/src/backend/executor/execTuples.c b/src/backend/executor/execTuples.c
index 37c46543cd..52692fc4f1 100644
--- a/src/backend/executor/execTuples.c
+++ b/src/backend/executor/execTuples.c
@@ -738,7 +738,7 @@ tts_buffer_heap_copyslot(TupleTableSlot *dstslot, TupleTableSlot *srcslot)
 		dstslot->tts_flags |= TTS_FLAG_SHOULDFREE;
 		dstslot->tts_flags &= ~TTS_FLAG_EMPTY;
 		oldContext = MemoryContextSwitchTo(dstslot->tts_mcxt);
-		bdstslot->base.tuple = ExecCopySlotHeapTuple(srcslot);;
+		bdstslot->base.tuple = ExecCopySlotHeapTuple(srcslot);
 		MemoryContextSwitchTo(oldContext);
 	}
 	else
diff --git a/src/backend/optimizer/prep/prepunion.c b/src/backend/optimizer/prep/prepunion.c
index 2a1c1cb2e1..da278f785e 100644
--- a/src/backend/optimizer/prep/prepunion.c
+++ b/src/backend/optimizer/prep/prepunion.c
@@ -817,7 +817,7 @@ generate_nonunion_paths(SetOperationStmt *op, PlannerInfo *root,
 	/* Build result relation. */
 	result_rel = fetch_upper_rel(root, UPPERREL_SETOP,
 								 bms_union(lrel->relids, rrel->relids));
-	result_rel->reltarget = create_pathtarget(root, tlist);;
+	result_rel->reltarget = create_pathtarget(root, tlist);
 
 	/*
 	 * Append the child results together.
#2Amit Kapila
amit.kapila16@gmail.com
In reply to: David Rowley (#1)
Re: Remove double trailing semicolons

On Mon, Dec 17, 2018 at 1:53 AM David Rowley
<david.rowley@2ndquadrant.com> wrote:

While looking over some recent commits, I noticed there are some code
lines with a double trailing semicolon instead of a single one. The
attached fixes these.

LGTM. I will commit it.

--
With Regards,
Amit Kapila.
EnterpriseDB: http://www.enterprisedb.com

#3Amit Kapila
amit.kapila16@gmail.com
In reply to: Amit Kapila (#2)
Re: Remove double trailing semicolons

On Mon, Dec 17, 2018 at 1:00 PM Amit Kapila <amit.kapila16@gmail.com> wrote:

On Mon, Dec 17, 2018 at 1:53 AM David Rowley
<david.rowley@2ndquadrant.com> wrote:

While looking over some recent commits, I noticed there are some code
lines with a double trailing semicolon instead of a single one.

Pushed, one of these goes back till 10.

--
With Regards,
Amit Kapila.
EnterpriseDB: http://www.enterprisedb.com

#4David Rowley
david.rowley@2ndquadrant.com
In reply to: Amit Kapila (#3)
Re: Remove double trailing semicolons

On Mon, 17 Dec 2018 at 22:10, Amit Kapila <amit.kapila16@gmail.com> wrote:

Pushed, one of these goes back till 10.

Thanks.

--
David Rowley http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training & Services