postgres.git / summary / log / commit / refs

commit    06a1ada7935acce6c6b9f5569ca3da9260a50784
Author:   Tom Lane <tgl@sss.pgh.pa.us>
Date:     Sat Sep 12 15:11:08 2015 +0000
First released in: 9.0.23, 9.1.19, 9.2.14

    Remove set-but-not-used variable.
    
    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.


 src/backend/optimizer/path/joinrels.c | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/src/backend/optimizer/path/joinrels.c b/src/backend/optimizer/path/joinrels.c
index 3666bf43910..fab8fd236a8 100644
--- a/src/backend/optimizer/path/joinrels.c
+++ b/src/backend/optimizer/path/joinrels.c
@@ -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
 		{


[parent: c5391e9334c3]