[COMMITTERS] pgsql: Improve castNode notation by introducinglist-extraction-specifi

Поиск
Список
Период
Сортировка
От Tom Lane
Тема [COMMITTERS] pgsql: Improve castNode notation by introducinglist-extraction-specifi
Дата
Msg-id E1cxdUL-0005Ta-8e@gemulon.postgresql.org
обсуждение исходный текст
Список pgsql-committers
Improve castNode notation by introducing list-extraction-specific variants.

This extends the castNode() notation introduced by commit 5bcab1114 to
provide, in one step, extraction of a list cell's pointer and coercion to
a concrete node type.  For example, "lfirst_node(Foo, lc)" is the same
as "castNode(Foo, lfirst(lc))".  Almost half of the uses of castNode
that have appeared so far include a list extraction call, so this is
pretty widely useful, and it saves a few more keystrokes compared to the
old way.

As with the previous patch, back-patch the addition of these macros to
pg_list.h, so that the notation will be available when back-patching.

Patch by me, after an idea of Andrew Gierth's.

Discussion: https://postgr.es/m/14197.1491841216@sss.pgh.pa.us

Branch
------
master

Details
-------
http://git.postgresql.org/pg/commitdiff/8f0530f58061b185dc385df42e62d78a18d4ae3e

Modified Files
--------------
contrib/pg_stat_statements/pg_stat_statements.c |  4 ++--
contrib/postgres_fdw/deparse.c                  |  2 +-
contrib/postgres_fdw/postgres_fdw.c             |  6 +++---
src/backend/catalog/objectaddress.c             | 16 ++++++++--------
src/backend/catalog/pg_proc.c                   |  2 +-
src/backend/commands/aggregatecmds.c            |  4 ++--
src/backend/commands/async.c                    |  8 ++++----
src/backend/commands/collationcmds.c            |  2 +-
src/backend/commands/copy.c                     |  6 +++---
src/backend/commands/createas.c                 |  2 +-
src/backend/commands/dropcmds.c                 |  8 ++++----
src/backend/commands/explain.c                  |  6 +++---
src/backend/commands/extension.c                |  4 ++--
src/backend/commands/foreigncmds.c              |  2 +-
src/backend/commands/functioncmds.c             |  5 +++--
src/backend/commands/matview.c                  |  2 +-
src/backend/commands/opclasscmds.c              |  6 +++---
src/backend/commands/portalcmds.c               |  2 +-
src/backend/commands/prepare.c                  |  4 ++--
src/backend/commands/tablecmds.c                |  4 ++--
src/backend/commands/trigger.c                  |  2 +-
src/backend/commands/user.c                     |  2 +-
src/backend/commands/view.c                     |  2 +-
src/backend/executor/execExpr.c                 |  2 +-
src/backend/executor/execTuples.c               |  2 +-
src/backend/executor/execUtils.c                |  2 +-
src/backend/executor/functions.c                |  8 ++++----
src/backend/executor/nodeAgg.c                  |  4 ++--
src/backend/executor/nodeHashjoin.c             |  2 +-
src/backend/executor/nodeLockRows.c             |  2 +-
src/backend/executor/nodeModifyTable.c          |  2 +-
src/backend/executor/nodeSubplan.c              |  8 ++++----
src/backend/executor/spi.c                      | 18 +++++++++---------
src/backend/nodes/nodeFuncs.c                   | 16 ++++++++--------
src/backend/optimizer/path/costsize.c           | 12 ++++++------
src/backend/optimizer/path/indxpath.c           |  4 ++--
src/backend/optimizer/path/joinrels.c           |  2 +-
src/backend/optimizer/plan/analyzejoins.c       |  2 +-
src/backend/optimizer/plan/createplan.c         |  8 ++++----
src/backend/optimizer/plan/planner.c            |  2 +-
src/backend/optimizer/plan/setrefs.c            |  2 +-
src/backend/optimizer/plan/subselect.c          |  2 +-
src/backend/optimizer/prep/prepunion.c          |  2 +-
src/backend/optimizer/util/clauses.c            |  2 +-
src/backend/optimizer/util/orclauses.c          |  2 +-
src/backend/optimizer/util/restrictinfo.c       |  6 +++---
src/backend/parser/analyze.c                    |  6 +++---
src/backend/parser/gram.y                       |  2 +-
src/backend/parser/parse_collate.c              |  8 ++++----
src/backend/parser/parse_expr.c                 |  4 ++--
src/backend/parser/parse_node.c                 |  2 +-
src/backend/parser/parse_relation.c             |  2 +-
src/backend/parser/parse_type.c                 |  4 ++--
src/backend/parser/parse_utilcmd.c              | 18 +++++++++---------
src/backend/rewrite/rewriteDefine.c             |  8 ++++----
src/backend/rewrite/rewriteHandler.c            | 10 +++++-----
src/backend/tcop/postgres.c                     | 12 ++++++------
src/backend/tcop/pquery.c                       |  6 +++---
src/backend/utils/adt/ruleutils.c               | 10 +++++-----
src/backend/utils/adt/selfuncs.c                |  2 +-
src/backend/utils/cache/plancache.c             | 22 +++++++++++-----------
src/backend/utils/misc/guc.c                    |  2 +-
src/backend/utils/mmgr/portalmem.c              |  2 +-
src/include/nodes/pg_list.h                     |  7 +++++++
src/pl/plpgsql/src/pl_exec.c                    |  4 ++--
65 files changed, 176 insertions(+), 168 deletions(-)


В списке pgsql-committers по дате отправления:

Предыдущее
От: Peter Eisentraut
Дата:
Сообщение: [COMMITTERS] pgsql: Support configuration reload in logical replication workers
Следующее
От: Andres Freund
Дата:
Сообщение: [COMMITTERS] pgsql: Fix initialization of dsa.c free area counter.