Add a parse location field to struct FunctionParameter.
authorTom Lane <[email protected]>
Thu, 31 Oct 2024 19:53:58 +0000 (15:53 -0400)
committerTom Lane <[email protected]>
Thu, 31 Oct 2024 20:09:27 +0000 (16:09 -0400)
commit89e51abcb2905d6beaa09a7d3603d6882ac27033
tree27dba1ec7f0bd023f1bfa5103fbafdf3c0841595
parentb82c877e76e2398409e823773413079668cf1881
Add a parse location field to struct FunctionParameter.

This allows an error cursor to be supplied for a bunch of
bad-function-definition errors that previously lacked one,
or that cheated a bit by pointing at the contained type name
when the error isn't really about that.

Bump catversion from an abundance of caution --- I don't think
this node type can actually appear in stored views/rules, but
better safe than sorry.

Jian He and Tom Lane (extracted from a larger  by Jian,
with some additional work by me)

Discussion: https://postgr.es/m/CACJufxEmONE3P2En=jopZy1m=cCCUs65M4+1o52MW5og9oaUPA@mail.gmail.com
17 files changed:
src/backend/commands/functioncmds.c
src/backend/nodes/nodeFuncs.c
src/backend/parser/gram.y
src/include/catalog/catversion.h
src/include/nodes/parsenodes.h
src/test/modules/test_ddl_deparse/expected/create_type.out
src/test/modules/test_ddl_deparse/expected/opfamily.out
src/test/modules/test_pg_dump/expected/test_pg_dump.out
src/test/regress/expected/create_cast.out
src/test/regress/expected/create_procedure.out
src/test/regress/expected/create_type.out
src/test/regress/expected/equivclass.out
src/test/regress/expected/expressions.out
src/test/regress/expected/float4-misrounded-input.out
src/test/regress/expected/float4.out
src/test/regress/expected/float8.out
src/test/regress/expected/polymorphism.out