Commit6844d32

committed

Remove useless if condition

We can call GetAttributeCompression() with a NULL argument. Ithandles that internally already. This change makes all the callers ofGetAttributeCompression() uniform.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 parent689c66a commit6844d32Copy full SHA for 6844d32

File tree

-5

lines changed

-5

lines changed

Lines changed: 1 addition & 5 deletions

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

Comments

(0)