diff options
| author | Peter Eisentraut <peter_e@gmx.net> | 2017-02-21 11:33:07 -0500 |
|---|---|---|
| committer | Peter Eisentraut <peter_e@gmx.net> | 2017-02-21 11:59:09 -0500 |
| commit | 38d103763d14baddf3cbfe4b00b501059fc9447f (patch) | |
| tree | a69106fc39af0e181307f40fd3ba43b1930c3fcc /src/backend/parser/parse_collate.c | |
| parent | 4e5ce3c1aeadf81b504bc9d683b67950bd3f8766 (diff) | |
| download | postgresql-38d103763d14baddf3cbfe4b00b501059fc9447f.tar.gz postgresql-38d103763d14baddf3cbfe4b00b501059fc9447f.zip | |
Make more use of castNode()
Diffstat (limited to 'src/backend/parser/parse_collate.c')
| -rw-r--r-- | src/backend/parser/parse_collate.c | 13 |
1 files changed, 4 insertions, 9 deletions
diff --git a/src/backend/parser/parse_collate.c b/src/backend/parser/parse_collate.c index 832897f68dd..52ac7227909 100644 --- a/src/backend/parser/parse_collate.c +++ b/src/backend/parser/parse_collate.c @@ -514,8 +514,7 @@ assign_collations_walker(Node *node, assign_collations_context *context) if (qtree->targetList == NIL) return false; - tent = (TargetEntry *) linitial(qtree->targetList); - Assert(IsA(tent, TargetEntry)); + tent = castNode(TargetEntry, linitial(qtree->targetList)); if (tent->resjunk) return false; @@ -650,9 +649,7 @@ assign_collations_walker(Node *node, assign_collations_context *context) foreach(lc, expr->args) { - CaseWhen *when = (CaseWhen *) lfirst(lc); - - Assert(IsA(when, CaseWhen)); + CaseWhen *when = castNode(CaseWhen, lfirst(lc)); /* * The condition expressions mustn't affect @@ -868,9 +865,8 @@ assign_aggregate_collations(Aggref *aggref, /* Process aggregated args, holding resjunk ones at arm's length */ foreach(lc, aggref->args) { - TargetEntry *tle = (TargetEntry *) lfirst(lc); + TargetEntry *tle = castNode(TargetEntry, lfirst(lc)); - Assert(IsA(tle, TargetEntry)); if (tle->resjunk) assign_expr_collations(loccontext->pstate, (Node *) tle); else @@ -913,9 +909,8 @@ assign_ordered_set_collations(Aggref *aggref, /* Process aggregated args appropriately */ foreach(lc, aggref->args) { - TargetEntry *tle = (TargetEntry *) lfirst(lc); + TargetEntry *tle = castNode(TargetEntry, lfirst(lc)); - Assert(IsA(tle, TargetEntry)); if (merge_sort_collations) (void) assign_collations_walker((Node *) tle, loccontext); else |