Commit3848d21

committed

Make memset() use sizeof() rather than re-compute size

This is simpler and more closely follows overwhelming precedent.Report and patch by Mark Dilger.Discussion:https://postgr.es/m/9A68FB88-5F45-4848-9926-8586E2D777D1@gmail.com

1 parent414cd43 commit3848d21Copy full SHA for 3848d21

File tree

-3

lines changed

-3

lines changed

Lines changed: 3 additions & 3 deletions

Original file line number	Diff line number	Diff line change
`@@ -301,9 +301,9 @@ statext_store(Relation pg_stext, Oid statOid,`
`301`	`301`	`boolnulls[Natts_pg_statistic_ext];`
`302`	`302`	`boolreplaces[Natts_pg_statistic_ext];`
`303`	`303`
`304`		`-memset(nulls,1,Natts_pg_statistic_ext*sizeof(bool));`
`305`		`-memset(replaces,0,Natts_pg_statistic_ext*sizeof(bool));`
`306`		`-memset(values,0,Natts_pg_statistic_ext*sizeof(Datum));`
	`304`	`+memset(nulls,true,sizeof(nulls));`
	`305`	`+memset(replaces,false,sizeof(replaces));`
	`306`	`+memset(values,0,sizeof(values));`
`307`	`307`
`308`	`308`	`/*`
`309`	`309`	`* Construct a new pg_statistic_ext tuple, replacing the calculated stats.`

Comments

(0)