Robert Haas <robertmh...@gmail.com> writes: > On Wed, Dec 20, 2017 at 6:06 PM, Tom Lane <t...@sss.pgh.pa.us> wrote: >> Anyway, I left it as-is, but I'm willing to make the change if >> people feel the other way is better.
> I feel the other way -- let's not add more pointer indirections if it > isn't really necessary. OK --- with any luck, the changes in the paramFetch API will force people to update relevant code anyway. It would only be an issue for code that is not calling paramFetch at all, and such code is broken anyway for dynamic param lists. Pushed without touching that issue. regards, tom lane