1berry/postgres
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Remove set-but-not-used variable.

Browse Source 
In branches before 9.3, commit 8703059c6 caused join_is_legal()'s
unique_ified variable to become unused, since its only remaining
use is for LATERAL-related tests which don't exist pre-9.3.
My compiler didn't complain about that, but Peter's does.
This commit is contained in:
Tom Lane 2015-09-12 11:11:08 -04:00
parent c5391e9334
commit 06a1ada793
1 changed files with 0 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/backend/optimizer/path/joinrels.c
View File

@ -324,7 +324,6 @@ join_is_legal(PlannerInfo *root, RelOptInfo *rel1, RelOptInfo *rel2,
{
SpecialJoinInfo *match_sjinfo;
bool reversed;
bool unique_ified;
bool must_be_leftjoin;
ListCell *l;
@ -342,7 +341,6 @@ join_is_legal(PlannerInfo *root, RelOptInfo *rel1, RelOptInfo *rel2,
*/
match_sjinfo = NULL;
reversed = false;
unique_ified = false;
must_be_leftjoin = false;
foreach(l, root->join_info_list)
@ -444,7 +442,6 @@ join_is_legal(PlannerInfo *root, RelOptInfo *rel1, RelOptInfo *rel2,
return false; /* invalid join path */
match_sjinfo = sjinfo;
reversed = false;
unique_ified = true;
}
else if (sjinfo->jointype == JOIN_SEMI &&
bms_equal(sjinfo->syn_righthand, rel1->relids) &&
@ -456,7 +453,6 @@ join_is_legal(PlannerInfo *root, RelOptInfo *rel1, RelOptInfo *rel2,
return false; /* invalid join path */
match_sjinfo = sjinfo;
reversed = true;
unique_ified = true;
}
else
{