diff options
| author | Peter Eisentraut <peter_e@gmx.net> | 2017-08-15 23:34:39 -0400 |
|---|---|---|
| committer | Peter Eisentraut <peter_e@gmx.net> | 2017-08-15 23:34:39 -0400 |
| commit | 77d05706beb115b412728bd94dce16d83795583d (patch) | |
| tree | d1154b7ff7b438d7ecd15b2d0bbd9c5b50086a18 /src/backend/utils/adt/ruleutils.c | |
| parent | 4d4c89171598424b31175ef9b600ac87a9a61023 (diff) | |
| download | postgresql-77d05706beb115b412728bd94dce16d83795583d.tar.gz postgresql-77d05706beb115b412728bd94dce16d83795583d.zip | |
Fix up some misusage of appendStringInfo() and friends
Change to appendStringInfoChar() or appendStringInfoString() where those
can be used.
Author: David Rowley <david.rowley@2ndquadrant.com>
Reviewed-by: Ashutosh Bapat <ashutosh.bapat@enterprisedb.com>
Diffstat (limited to 'src/backend/utils/adt/ruleutils.c')
| -rw-r--r-- | src/backend/utils/adt/ruleutils.c | 20 |
1 files changed, 10 insertions, 10 deletions
diff --git a/src/backend/utils/adt/ruleutils.c b/src/backend/utils/adt/ruleutils.c index e9bd64b7a88..7469ec773c6 100644 --- a/src/backend/utils/adt/ruleutils.c +++ b/src/backend/utils/adt/ruleutils.c @@ -1656,11 +1656,11 @@ pg_get_partkeydef_worker(Oid relid, int prettyFlags, { case PARTITION_STRATEGY_LIST: if (!attrsOnly) - appendStringInfo(&buf, "LIST"); + appendStringInfoString(&buf, "LIST"); break; case PARTITION_STRATEGY_RANGE: if (!attrsOnly) - appendStringInfo(&buf, "RANGE"); + appendStringInfoString(&buf, "RANGE"); break; default: elog(ERROR, "unexpected partition strategy: %d", @@ -1668,7 +1668,7 @@ pg_get_partkeydef_worker(Oid relid, int prettyFlags, } if (!attrsOnly) - appendStringInfo(&buf, " ("); + appendStringInfoString(&buf, " ("); sep = ""; for (keyno = 0; keyno < form->partnatts; keyno++) { @@ -5635,10 +5635,10 @@ get_rule_sortgroupclause(Index ref, List *tlist, bool force_colno, ||IsA(expr, WindowFunc)); if (need_paren) - appendStringInfoString(context->buf, "("); + appendStringInfoChar(context->buf, '('); get_rule_expr(expr, context, true); if (need_paren) - appendStringInfoString(context->buf, ")"); + appendStringInfoChar(context->buf, ')'); } return expr; @@ -5665,7 +5665,7 @@ get_rule_groupingset(GroupingSet *gset, List *targetlist, case GROUPING_SET_SIMPLE: { if (!omit_parens || list_length(gset->content) != 1) - appendStringInfoString(buf, "("); + appendStringInfoChar(buf, '('); foreach(l, gset->content) { @@ -5678,7 +5678,7 @@ get_rule_groupingset(GroupingSet *gset, List *targetlist, } if (!omit_parens || list_length(gset->content) != 1) - appendStringInfoString(buf, ")"); + appendStringInfoChar(buf, ')'); } return; @@ -5701,7 +5701,7 @@ get_rule_groupingset(GroupingSet *gset, List *targetlist, sep = ", "; } - appendStringInfoString(buf, ")"); + appendStringInfoChar(buf, ')'); } /* @@ -8713,7 +8713,7 @@ get_rule_expr(Node *node, deparse_context *context, sep = ", "; } - appendStringInfoString(buf, ")"); + appendStringInfoChar(buf, ')'); break; case PARTITION_STRATEGY_RANGE: @@ -10941,7 +10941,7 @@ get_range_partbound_string(List *bound_datums) } sep = ", "; } - appendStringInfoString(buf, ")"); + appendStringInfoChar(buf, ')'); return buf->data; } |