Commit3fd80c7

committed

Narrow the scope of a local variable.

This is better style and more symmetrical with the other if-branch.This likely should have been included in9de77b5 (which createdthe opportunity), but it was overlooked.Japin LiDiscussion:https://postgr.es/m/MEYP282MB16699FA4A7CD57EB250E871FB6A40@MEYP282MB1669.AUSP282.PROD.OUTLOOK.COM

1 parenta6cf3df commit3fd80c7Copy full SHA for 3fd80c7

File tree

-1

lines changed

-1

lines changed

Lines changed: 2 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -493,7 +493,6 @@ logicalrep_write_tuple(StringInfo out, Relation rel, HeapTuple tuple, bool binar`
`493`	`493`	`HeapTupletyptup;`
`494`	`494`	`Form_pg_typetypclass;`
`495`	`495`	`Form_pg_attributeatt=TupleDescAttr(desc,i);`
`496`		`-char*outputstr;`
`497`	`496`
`498`	`497`	`if (att->attisdropped\|\|att->attgenerated)`
`499`	`498`	`continue;`
`@@ -537,6 +536,8 @@ logicalrep_write_tuple(StringInfo out, Relation rel, HeapTuple tuple, bool binar`
`537`	`536`	`}`
`538`	`537`	`else`
`539`	`538`	`{`
	`539`	`+char*outputstr;`
	`540`	`+`
`540`	`541`	`pq_sendbyte(out,LOGICALREP_COLUMN_TEXT);`
`541`	`542`	`outputstr=OidOutputFunctionCall(typclass->typoutput,values[i]);`
`542`	`543`	`pq_sendcountedtext(out,outputstr,strlen(outputstr), false);`

Comments

(0)