diff options
author | Peter Eisentraut <peter@eisentraut.org> | 2019-03-14 13:30:09 +0100 |
---|---|---|
committer | Peter Eisentraut <peter@eisentraut.org> | 2019-03-14 13:30:09 +0100 |
commit | c6ff0b892c30122b75d32c524109d16ee3c973f0 (patch) | |
tree | c01cf8fe3258deb6a28af5e7d28a4ea81eda6077 /src/backend/executor/functions.c | |
parent | 1226d932b4dadc39ba2f9a488e4d784443ea6a78 (diff) | |
download | postgresql-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.c | 15 |
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 = ¶mLI->params[i]; |