aboutsummaryrefslogtreecommitdiff
path: root/src/backend/executor/functions.c
diff options
context:
space:
mode:
authorPeter Eisentraut <peter@eisentraut.org>2019-03-14 13:30:09 +0100
committerPeter Eisentraut <peter@eisentraut.org>2019-03-14 13:30:09 +0100
commitc6ff0b892c30122b75d32c524109d16ee3c973f0 (patch)
treec01cf8fe3258deb6a28af5e7d28a4ea81eda6077 /src/backend/executor/functions.c
parent1226d932b4dadc39ba2f9a488e4d784443ea6a78 (diff)
downloadpostgresql-c6ff0b892c30122b75d32c524109d16ee3c973f0.tar.gz
postgresql-c6ff0b892c30122b75d32c524109d16ee3c973f0.zip
Refactor ParamListInfo initialization
There were six copies of identical nontrivial code. Put it into a function.
Diffstat (limited to 'src/backend/executor/functions.c')
-rw-r--r--src/backend/executor/functions.c15
1 files changed, 2 insertions, 13 deletions
diff --git a/src/backend/executor/functions.c b/src/backend/executor/functions.c
index c6b7203f813..965e5dea70e 100644
--- a/src/backend/executor/functions.c
+++ b/src/backend/executor/functions.c
@@ -906,21 +906,10 @@ postquel_sub_params(SQLFunctionCachePtr fcache,
if (nargs > 0)
{
ParamListInfo paramLI;
- int i;
if (fcache->paramLI == NULL)
{
- paramLI = (ParamListInfo)
- palloc(offsetof(ParamListInfoData, params) +
- nargs * sizeof(ParamExternData));
- /* we have static list of params, so no hooks needed */
- paramLI->paramFetch = NULL;
- paramLI->paramFetchArg = NULL;
- paramLI->paramCompile = NULL;
- paramLI->paramCompileArg = NULL;
- paramLI->parserSetup = NULL;
- paramLI->parserSetupArg = NULL;
- paramLI->numParams = nargs;
+ paramLI = makeParamList(nargs);
fcache->paramLI = paramLI;
}
else
@@ -929,7 +918,7 @@ postquel_sub_params(SQLFunctionCachePtr fcache,
Assert(paramLI->numParams == nargs);
}
- for (i = 0; i < nargs; i++)
+ for (int i = 0; i < nargs; i++)
{
ParamExternData *prm = &paramLI->params[i];