From 1c5f223e255fdb128d01ad40be629fd9ae8713e5 Mon Sep 17 00:00:00 2001 From: Tom Lane Date: Mon, 24 Nov 2003 16:54:07 +0000 Subject: Overdue code review for ALTER SEQUENCE patch. Don't generate illegal Node tree for CYCLE option; don't assume zeros are invalid values for sequence fields other than increment_by; don't reset cache_value when not told to; simplify code for testing whether to apply defaults. --- src/backend/parser/gram.y | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'src/backend/parser') diff --git a/src/backend/parser/gram.y b/src/backend/parser/gram.y index 337eef4480c..3452b50fb03 100644 --- a/src/backend/parser/gram.y +++ b/src/backend/parser/gram.y @@ -11,7 +11,7 @@ * * * IDENTIFICATION - * $Header: /cvsroot/pgsql/src/backend/parser/gram.y,v 2.438 2003/11/21 22:32:49 tgl Exp $ + * $Header: /cvsroot/pgsql/src/backend/parser/gram.y,v 2.439 2003/11/24 16:54:07 tgl Exp $ * * HISTORY * AUTHOR DATE MAJOR EVENT @@ -1926,11 +1926,11 @@ OptSeqElem: CACHE NumericOnly } | CYCLE { - $$ = makeDefElem("cycle", (Node *)true); + $$ = makeDefElem("cycle", (Node *)makeInteger(TRUE)); } | NO CYCLE { - $$ = makeDefElem("cycle", (Node *)false); + $$ = makeDefElem("cycle", (Node *)makeInteger(FALSE)); } | INCREMENT opt_by NumericOnly { -- cgit v1.2.3