diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2016-11-23 16:26:40 -0500 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2016-11-23 16:26:40 -0500 |
| commit | 6fa391be4e83139cd134d5ccfc1499809bb8c98c (patch) | |
| tree | 39eb6df2b09b4c0fbfab88830e32822d5779c589 /src | |
| parent | bd673e8e864a1987eca8d40c593e857ab8d0a5ba (diff) | |
| download | postgresql-6fa391be4e83139cd134d5ccfc1499809bb8c98c.tar.gz postgresql-6fa391be4e83139cd134d5ccfc1499809bb8c98c.zip | |
Avoid masking a function parameter name with a local variable name.
No actual bug here, but it might confuse readers, so change the name
of the local variable.
Ashutosh Bapat
Diffstat (limited to 'src')
| -rw-r--r-- | src/backend/optimizer/path/joinpath.c | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/src/backend/optimizer/path/joinpath.c b/src/backend/optimizer/path/joinpath.c index cc7384f7e5e..e47189a339d 100644 --- a/src/backend/optimizer/path/joinpath.c +++ b/src/backend/optimizer/path/joinpath.c @@ -131,7 +131,7 @@ add_paths_to_joinrel(PlannerInfo *root, */ foreach(lc, root->join_info_list) { - SpecialJoinInfo *sjinfo = (SpecialJoinInfo *) lfirst(lc); + SpecialJoinInfo *sjinfo2 = (SpecialJoinInfo *) lfirst(lc); /* * SJ is relevant to this join if we have some part of its RHS @@ -140,19 +140,19 @@ add_paths_to_joinrel(PlannerInfo *root, * join has already been proven legal.) If the SJ is relevant, it * presents constraints for joining to anything not in its RHS. */ - if (bms_overlap(joinrel->relids, sjinfo->min_righthand) && - !bms_overlap(joinrel->relids, sjinfo->min_lefthand)) + if (bms_overlap(joinrel->relids, sjinfo2->min_righthand) && + !bms_overlap(joinrel->relids, sjinfo2->min_lefthand)) extra.param_source_rels = bms_join(extra.param_source_rels, bms_difference(root->all_baserels, - sjinfo->min_righthand)); + sjinfo2->min_righthand)); /* full joins constrain both sides symmetrically */ - if (sjinfo->jointype == JOIN_FULL && - bms_overlap(joinrel->relids, sjinfo->min_lefthand) && - !bms_overlap(joinrel->relids, sjinfo->min_righthand)) + if (sjinfo2->jointype == JOIN_FULL && + bms_overlap(joinrel->relids, sjinfo2->min_lefthand) && + !bms_overlap(joinrel->relids, sjinfo2->min_righthand)) extra.param_source_rels = bms_join(extra.param_source_rels, bms_difference(root->all_baserels, - sjinfo->min_lefthand)); + sjinfo2->min_lefthand)); } /* |