diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2002-12-12 20:35:16 +0000 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2002-12-12 20:35:16 +0000 |
| commit | b0422b215c100bb29c1071872012a1f3c6681058 (patch) | |
| tree | a764deb63dc546abe6c0b283feac110fd22d01c4 /src/backend/parser/parse_coerce.c | |
| parent | ff7349694f399d0063b51419ea6e25770334f363 (diff) | |
| download | postgresql-b0422b215c100bb29c1071872012a1f3c6681058.tar.gz | |
Preliminary code review for domain CHECK constraints patch: add documentation,
make VALUE a non-reserved word again, use less invasive method of passing
ConstraintTestValue into transformExpr, fix problems with nested constraint
testing, do correct thing with NULL result from a constraint expression,
remove memory leak. Domain checks still need much more work if we are going
to allow ALTER DOMAIN, however.
Diffstat (limited to 'src/backend/parser/parse_coerce.c')
| -rw-r--r-- | src/backend/parser/parse_coerce.c | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/src/backend/parser/parse_coerce.c b/src/backend/parser/parse_coerce.c index 3856939d40..af184eca6d 100644 --- a/src/backend/parser/parse_coerce.c +++ b/src/backend/parser/parse_coerce.c @@ -8,7 +8,7 @@ * * * IDENTIFICATION - * $Header: /cvsroot/pgsql/src/backend/parser/parse_coerce.c,v 2.90 2002/12/12 15:49:38 tgl Exp $ + * $Header: /cvsroot/pgsql/src/backend/parser/parse_coerce.c,v 2.91 2002/12/12 20:35:13 tgl Exp $ * *------------------------------------------------------------------------- */ @@ -458,9 +458,8 @@ coerce_type_constraints(Node *arg, Oid typeId, CoercionForm cformat) r->testtype = CONSTR_TEST_CHECK; r->name = NameStr(c->conname); r->domname = NameStr(typTup->typname); - r->check_expr = stringToNode(MemoryContextStrdup(CacheMemoryContext, - DatumGetCString(DirectFunctionCall1(textout, - val)))); + r->check_expr = stringToNode(DatumGetCString(DirectFunctionCall1(textout, + val))); arg = (Node *) r; } |