pgsql: Move strip_implicit_coercions() from optimizer to nodeFuncs.c. - Mailing list pgsql-committers

From Tom Lane
Subject pgsql: Move strip_implicit_coercions() from optimizer to nodeFuncs.c.
Date
Msg-id E1V1kxb-0006mj-2L@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Move strip_implicit_coercions() from optimizer to nodeFuncs.c.

Use of this function has spread into the parser and rewriter, so it seems
like time to pull it out of the optimizer and put it into the more central
nodeFuncs module.  This eliminates the need to #include optimizer/clauses.h
in most of the calling files, demonstrating that this function was indeed a
bit outside the normal code reference patterns.

Branch
------
master

Details
-------
http://git.postgresql.org/pg/commitdiff/10a509d82956dee14eb2011bd266cd3c728ae188

Modified Files
--------------
src/backend/nodes/nodeFuncs.c        |   59 ++++++++++++++++++++++++++++++++++
src/backend/optimizer/util/clauses.c |   56 --------------------------------
src/backend/parser/parse_clause.c    |    1 -
src/backend/parser/parse_relation.c  |    1 -
src/backend/rewrite/rewriteHandler.c |    1 -
src/backend/utils/adt/ruleutils.c    |    1 -
src/include/nodes/nodeFuncs.h        |    1 +
src/include/optimizer/clauses.h      |    2 --
8 files changed, 60 insertions(+), 62 deletions(-)


pgsql-committers by date:

Previous
From: Tom Lane
Date:
Subject: pgsql: Further hacking on ruleutils' new column-alias-assignment code.
Next
From: Tom Lane
Date:
Subject: pgsql: Fix booltestsel() for case where we have NULL stats but not MCV