diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2008-12-19 05:04:35 +0000 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2008-12-19 05:04:35 +0000 |
| commit | adac22bf8a451eef87ff5693cde021e738191c24 (patch) | |
| tree | e6e7b8e5e2f86d67af1c7d476ac38d2af1289cf2 /src/backend/utils/adt/ruleutils.c | |
| parent | e32014d85de478e7baa4af9a113832ea1cd360ef (diff) | |
| download | postgresql-adac22bf8a451eef87ff5693cde021e738191c24.tar.gz postgresql-adac22bf8a451eef87ff5693cde021e738191c24.zip | |
When we added the ability to have zero-element ARRAY[] constructs by adding an
explicit cast to show the intended array type, we forgot to teach ruleutils.c
to print out such constructs properly. Found by noting bogus output from
recent changes in polymorphism regression test.
Diffstat (limited to 'src/backend/utils/adt/ruleutils.c')
| -rw-r--r-- | src/backend/utils/adt/ruleutils.c | 10 |
1 files changed, 9 insertions, 1 deletions
diff --git a/src/backend/utils/adt/ruleutils.c b/src/backend/utils/adt/ruleutils.c index 5a6540b88ea..444ee7a2007 100644 --- a/src/backend/utils/adt/ruleutils.c +++ b/src/backend/utils/adt/ruleutils.c @@ -9,7 +9,7 @@ * * * IDENTIFICATION - * $PostgreSQL: pgsql/src/backend/utils/adt/ruleutils.c,v 1.289 2008/12/18 18:20:34 tgl Exp $ + * $PostgreSQL: pgsql/src/backend/utils/adt/ruleutils.c,v 1.290 2008/12/19 05:04:35 tgl Exp $ * *------------------------------------------------------------------------- */ @@ -4451,6 +4451,14 @@ get_rule_expr(Node *node, deparse_context *context, appendStringInfo(buf, "ARRAY["); get_rule_expr((Node *) arrayexpr->elements, context, true); appendStringInfoChar(buf, ']'); + /* + * If the array isn't empty, we assume its elements are + * coerced to the desired type. If it's empty, though, we + * need an explicit coercion to the array type. + */ + if (arrayexpr->elements == NIL) + appendStringInfo(buf, "::%s", + format_type_with_typemod(arrayexpr->array_typeid, -1)); } break; |