Fix oversight in CALL argument handling, and do some minor cleanup.
authorTom Lane <tgl@sss.pgh.pa.us>
Sat, 10 Feb 2018 18:05:14 +0000 (13:05 -0500)
committerTom Lane <tgl@sss.pgh.pa.us>
Sat, 10 Feb 2018 18:05:14 +0000 (13:05 -0500)
commit65b1d767856d96c7d6f952f30890dd5b7d4b66bb
tree81f43c2ee257f223e7d9f7f8943ac5274e24bb16
parentfad15f4a547ad433a28c370bd071b08df9e65f10
Fix oversight in CALL argument handling, and do some minor cleanup.

CALL statements cannot support sub-SELECTs in the arguments of the called
procedure, since they just use ExecEvalExpr to evaluate such arguments.
Teach transformSubLink() to reject the case, as it already does for other
contexts in which subqueries are not supported.

In passing, s/EXPR_KIND_CALL/EXPR_KIND_CALL_ARGUMENT/ to make that enum
symbol line up more closely with the phrasing of the error messages it is
associated with.  And fix someone's weak grasp of English grammar in the
preceding EXPR_KIND_PARTITION_EXPRESSION addition.  Also update an
incorrect comment in resolve_unique_index_expr (possibly it was correct
when written, but nowadays transformExpr definitely does reject SRFs here).

Per report from Pavel Stehule --- but this resolves only one of the bugs
he mentions.

Discussion: https://postgr.es/m/CAFj8pRDxOwPPzpA8i+AQeDQFj7bhVw-dR2==rfWZ3zMGkm568Q@mail.gmail.com
src/backend/commands/functioncmds.c
src/backend/parser/parse_agg.c
src/backend/parser/parse_clause.c
src/backend/parser/parse_expr.c
src/backend/parser/parse_func.c
src/include/parser/parse_node.h
src/test/regress/expected/create_table.out