[COMMITTERS] pgsql: Remove no-longer-needed loop in ExecGather(). - Mailing list pgsql-committers

From Tom Lane
Subject [COMMITTERS] pgsql: Remove no-longer-needed loop in ExecGather().
Date
Msg-id E1cVLIv-0001Su-36@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Remove no-longer-needed loop in ExecGather().

Coverity complained quite properly that commit ea15e1867 had introduced
unreachable code into ExecGather(); to wit, it was no longer possible to
iterate the final for-loop more or less than once.  So remove the for().

In passing, clean up a couple of comments, and make better use of a local
variable.

Branch
------
master

Details
-------
http://git.postgresql.org/pg/commitdiff/0a8b9d3b2c57028f7100078cd711370f396d5a81

Modified Files
--------------
src/backend/executor/nodeGather.c | 39 +++++++++++++++------------------------
1 file changed, 15 insertions(+), 24 deletions(-)


pgsql-committers by date:

Previous
From: Peter Eisentraut
Date:
Subject: [COMMITTERS] pgsql: Add missing break
Next
From: Tom Lane
Date:
Subject: [COMMITTERS] pgsql: Relocate static function declarations to be after typedefs injs