diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2020-09-04 14:32:10 -0400 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2020-09-04 14:32:19 -0400 |
| commit | 38a2d703298c9a891dc9c24c0c087f417f555c70 (patch) | |
| tree | 5c8dfa0662efa40432bf05f7f2029c2ad198bac4 /src/backend/optimizer/plan/planner.c | |
| parent | 3eb3d3e7822d5eecfcaba871a90263c6025c5216 (diff) | |
| download | postgresql-38a2d703298c9a891dc9c24c0c087f417f555c70.tar.gz postgresql-38a2d703298c9a891dc9c24c0c087f417f555c70.zip | |
Remove some more useless assignments.
Found with clang's scan-build tool. It also whines about a lot of
other dead stores that we should *not* change IMO, either as a matter
of style or future-proofing. But these places seem like clear
oversights.
Discussion: https://postgr.es/m/CAEudQAo1+AcGppxDSg8k+zF4+Kv+eJyqzEDdbpDg58-=MQcerQ@mail.gmail.com
Diffstat (limited to 'src/backend/optimizer/plan/planner.c')
| -rw-r--r-- | src/backend/optimizer/plan/planner.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/backend/optimizer/plan/planner.c b/src/backend/optimizer/plan/planner.c index b40a112c25b..139c5e3dc24 100644 --- a/src/backend/optimizer/plan/planner.c +++ b/src/backend/optimizer/plan/planner.c @@ -5097,7 +5097,7 @@ create_ordered_paths(PlannerInfo *root, foreach(lc, input_rel->partial_pathlist) { Path *input_path = (Path *) lfirst(lc); - Path *sorted_path = input_path; + Path *sorted_path; bool is_sorted; int presorted_keys; double total_groups; |