diff options
| author | Tom Lane | 2017-01-27 03:09:34 +0000 |
|---|---|---|
| committer | Tom Lane | 2017-01-27 03:09:34 +0000 |
| commit | 7afd56c3c6d8360a5bfdfb2de30038b239fd756b (patch) | |
| tree | df9eb70bc951cdfe35629861285d5c6f31789ad7 /src/backend/rewrite | |
| parent | 9ba8a9ce4548bb34b7136b7463a61b2c499979a3 (diff) | |
Use castNode() in a bunch of statement-list-related code.
When I wrote commit ab1f0c822, I really missed the castNode() macro that
Peter E. had proposed shortly before. This back-fills the uses I would
have put it to. It's probably not all that significant, but there are
more assertions here than there were before, and conceivably they will
help catch any bugs associated with those representation changes.
I left behind a number of usages like "(Query *) copyObject(query_var)".
Those could have been converted as well, but Peter has proposed another
notational improvement that would handle copyObject cases automatically,
so I let that be for now.
Diffstat (limited to 'src/backend/rewrite')
| -rw-r--r-- | src/backend/rewrite/rewriteDefine.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/src/backend/rewrite/rewriteDefine.c b/src/backend/rewrite/rewriteDefine.c index 864d45ff12..481868bf53 100644 --- a/src/backend/rewrite/rewriteDefine.c +++ b/src/backend/rewrite/rewriteDefine.c @@ -173,7 +173,7 @@ InsertRule(char *rulname, if (event_qual != NULL) { /* Find query containing OLD/NEW rtable entries */ - Query *qry = (Query *) linitial(action); + Query *qry = castNode(Query, linitial(action)); qry = getInsertSelectQuery(qry, NULL); recordDependencyOnExpr(&myself, event_qual, qry->rtable, @@ -286,7 +286,7 @@ DefineQueryRewrite(char *rulename, */ foreach(l, action) { - query = (Query *) lfirst(l); + query = castNode(Query, lfirst(l)); if (query->resultRelation == 0) continue; /* Don't be fooled by INSERT/SELECT */ @@ -328,7 +328,7 @@ DefineQueryRewrite(char *rulename, /* * ... the one action must be a SELECT, ... */ - query = (Query *) linitial(action); + query = castNode(Query, linitial(action)); if (!is_instead || query->commandType != CMD_SELECT) ereport(ERROR, @@ -482,7 +482,7 @@ DefineQueryRewrite(char *rulename, foreach(l, action) { - query = (Query *) lfirst(l); + query = castNode(Query, lfirst(l)); if (!query->returningList) continue; @@ -813,7 +813,7 @@ setRuleCheckAsUser_Query(Query *qry, Oid userid) { CommonTableExpr *cte = (CommonTableExpr *) lfirst(l); - setRuleCheckAsUser_Query((Query *) cte->ctequery, userid); + setRuleCheckAsUser_Query(castNode(Query, cte->ctequery), userid); } /* If there are sublinks, search for them and process their RTEs */ |