]> granicus.if.org Git - postgresql/commit
In plpgsql, unify duplicate variables for record and row cases.
authorTom Lane <tgl@sss.pgh.pa.us>
Fri, 8 Dec 2017 16:20:50 +0000 (11:20 -0500)
committerTom Lane <tgl@sss.pgh.pa.us>
Fri, 8 Dec 2017 16:20:58 +0000 (11:20 -0500)
commitdd759b96ea8fa41b48541dd321c07d9a947f8de9
tree618deaf831f511a878146eae9937aa46cd8ef4c6
parent2d2d06b7e27e3177d5bef0061801c75946871db3
In plpgsql, unify duplicate variables for record and row cases.

plpgsql's function exec_move_row() handles assignment of a composite
source value to either a PLpgSQL_rec or PLpgSQL_row target variable.
Oddly, rather than taking a single target argument which it could do
run-time type detection on, it was coded to take two separate arguments
(only one of which is allowed to be non-NULL).  This choice had then
back-propagated into storing two separate target variables in various
plpgsql statement nodes, with lots of duplicative coding and awkward
interface logic to support that.  Simplify matters by folding those
pairs down to single variables, distinguishing the two cases only
where we must ... which turns out to be only in exec_move_row itself.
This is purely refactoring and should not change any behavior.

In passing, remove unused field PLpgSQL_stmt_open.returntype.

Discussion: https://postgr.es/m/11787.1512713374@sss.pgh.pa.us
src/pl/plpgsql/src/pl_comp.c
src/pl/plpgsql/src/pl_exec.c
src/pl/plpgsql/src/pl_funcs.c
src/pl/plpgsql/src/pl_gram.y
src/pl/plpgsql/src/plpgsql.h