Andrey Borodin <x4...@yandex-team.ru> writes:
>> On 3 Feb 2025, at 22:36, Tom Lane <t...@sss.pgh.pa.us> wrote:
>> I'm not wedded to that name; do you have a better idea?
> I'd propose something like attached. But feel free to ignore my suggestion: I
> do not understand context of these structure members.
Hmm, you're suggesting naming those field members after PL/pgSQL's
specific use of them. But the intent was that they are generic
workspace for anything that provides a EEOP_PARAM_CALLBACK
callback --- that is, the "param" in the field name refers to the
fact that this is an expression step for some kind of Param, and
not to what PL/pgSQL happens to do with the field.
Admittedly this is all moot unless some other extension starts
using EEOP_PARAM_CALLBACK, and I didn't find any evidence of that
using Debian Code Search. But I don't want to think of
EEOP_PARAM_CALLBACK as being specifically tied to PL/pgSQL.
regards, tom lane
