diff options
| author | Tom Lane | 2015-09-12 15:11:08 +0000 |
|---|---|---|
| committer | Tom Lane | 2015-09-12 15:11:08 +0000 |
| commit | 06a1ada7935acce6c6b9f5569ca3da9260a50784 (patch) | |
| tree | 13201eba87e9081878cc9b82e6aedd347bc34d76 | |
| parent | c5391e9334c32fb2cb9251f0e7bb824bb50c9d8c (diff) | |
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.
| -rw-r--r-- | src/backend/optimizer/path/joinrels.c | 4 |
1 files changed, 0 insertions, 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 { |