mirrors/postgres
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Remove redundant initialization of a local variable.

Browse Source 
In what was doubtless a typo, commit bf6c614a2 introduced a duplicate
initialization of a local variable.  This made Coverity unhappy, as well
as pretty much anybody reading the code.  We don't even have a real use
for the local variable, so just remove it.
This commit is contained in:
Tom Lane 2018-02-18 23:32:56 -05:00
parent ebf6049ebe
commit 8c44802b6e
1 changed files with 1 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
src/backend/executor/nodeGroup.c
View File

@ -162,7 +162,6 @@ GroupState *
ExecInitGroup(Group *node, EState *estate, int eflags) ExecInitGroup(Group *node, EState *estate, int eflags)
{ {
GroupState *grpstate; GroupState *grpstate;
AttrNumber *grpColIdx = grpColIdx = node->grpColIdx;
/* check for unsupported flags */ /* check for unsupported flags */
Assert(!(eflags & (EXEC_FLAG_BACKWARD | EXEC_FLAG_MARK))); Assert(!(eflags & (EXEC_FLAG_BACKWARD | EXEC_FLAG_MARK)));
@ -209,7 +208,7 @@ ExecInitGroup(Group *node, EState *estate, int eflags)
grpstate->eqfunction = grpstate->eqfunction =
execTuplesMatchPrepare(ExecGetResultType(outerPlanState(grpstate)), execTuplesMatchPrepare(ExecGetResultType(outerPlanState(grpstate)),
node->numCols, node->numCols,
grpColIdx, node->grpColIdx,
node->grpOperators, node->grpOperators,
&grpstate->ss.ps); &grpstate->ss.ps);