Commit1f44a31

committed

Add missing reset of need_initialization in reloptions code.

This resulted in useless extra work during every call of parseRelOptions,but no bad effects other than that. Noted by Alvaro.

1 parent924d6ed commit1f44a31Copy full SHA for 1f44a31

File tree

-2

lines changed

-2

lines changed

Lines changed: 6 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -8,7 +8,7 @@`
`8`	`8`	`*`
`9`	`9`	`*`
`10`	`10`	`* IDENTIFICATION`
`11`		`- * $PostgreSQL: pgsql/src/backend/access/common/reloptions.c,v 1.33 2010/02/26 02:00:32 momjian Exp $`
	`11`	`+ * $PostgreSQL: pgsql/src/backend/access/common/reloptions.c,v 1.34 2010/03/11 21:47:19 tgl Exp $`
`12`	`12`	`*`
`13`	`13`	`*-------------------------------------------------------------------------`
`14`	`14`	`*/`
`@@ -244,8 +244,9 @@ static void`
`244`	`244`	`initialize_reloptions(void)`
`245`	`245`	`{`
`246`	`246`	`inti;`
`247`		`-intj=0;`
	`247`	`+intj;`
`248`	`248`
	`249`	`+j=0;`
`249`	`250`	`for (i=0;boolRelOpts[i].gen.name;i++)`
`250`	`251`	`j++;`
`251`	`252`	`for (i=0;intRelOpts[i].gen.name;i++)`
`@@ -302,6 +303,9 @@ initialize_reloptions(void)`
`302`	`303`
`303`	`304`	`/* add a list terminator */`
`304`	`305`	`relOpts[j]=NULL;`
	`306`	`+`
	`307`	`+/* flag the work is complete */`
	`308`	`+need_initialization= false;`
`305`	`309`	`}`
`306`	`310`
`307`	`311`	`/*`

Comments

(0)