diff options
| author | Tom Lane | 2010-03-11 21:47:25 +0000 |
|---|---|---|
| committer | Tom Lane | 2010-03-11 21:47:25 +0000 |
| commit | eb9954e362127c582efe22e99f7b6c2db9d95dab (patch) | |
| tree | b187fb38f30aa0a00056ad7fdc169273e0bebd52 | |
| parent | 2875cd24db5109a3b84a07f05d4e61f576d47ec1 (diff) | |
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.
| -rw-r--r-- | src/backend/access/common/reloptions.c | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/src/backend/access/common/reloptions.c b/src/backend/access/common/reloptions.c index e1ce8596132..83c41b3da41 100644 --- a/src/backend/access/common/reloptions.c +++ b/src/backend/access/common/reloptions.c @@ -8,7 +8,7 @@ * * * IDENTIFICATION - * $PostgreSQL: pgsql/src/backend/access/common/reloptions.c,v 1.28.2.1 2009/08/27 17:19:31 alvherre Exp $ + * $PostgreSQL: pgsql/src/backend/access/common/reloptions.c,v 1.28.2.2 2010/03/11 21:47:25 tgl Exp $ * *------------------------------------------------------------------------- */ @@ -210,8 +210,9 @@ static void initialize_reloptions(void) { int i; - int j = 0; + int j; + j = 0; for (i = 0; boolRelOpts[i].gen.name; i++) j++; for (i = 0; intRelOpts[i].gen.name; i++) @@ -268,6 +269,9 @@ initialize_reloptions(void) /* add a list terminator */ relOpts[j] = NULL; + + /* flag the work is complete */ + need_initialization = false; } /* |