Commit689c66a

committed

Remove useless if condition

This is useless because these fields are not set anywhere before, sowe can assign them unconditionally. This also makes this moreconsistent with ATExecAddColumn().Reviewed-by: Alvaro Herrera <alvherre@alvh.no-ip.org>Discussion:https://www.postgresql.org/message-id/flat/52a125e4-ff9a-95f5-9f61-b87cf447e4da@eisentraut.org

1 parent1fa9241 commit689c66aCopy full SHA for 689c66a

File tree

-5

lines changed

-5

lines changed

Lines changed: 2 additions & 5 deletions

Original file line number	Diff line number	Diff line change
`@@ -942,11 +942,8 @@ DefineRelation(CreateStmt *stmt, char relkind, Oid ownerId,`
`942`	`942`	`attr->atthasdef = true;`
`943`	`943`	`}`
`944`	`944`
`945`		`-if (colDef->identity)`
`946`		`-attr->attidentity = colDef->identity;`
`947`		`-`
`948`		`-if (colDef->generated)`
`949`		`-attr->attgenerated = colDef->generated;`
	`945`	`+attr->attidentity = colDef->identity;`
	`946`	`+attr->attgenerated = colDef->generated;`
`950`	`947`
`951`	`948`	`if (colDef->compression)`
`952`	`949`	`attr->attcompression = GetAttributeCompression(attr->atttypid,`

Comments

(0)