diff options
| author | Matthias Krüger <matthias.krueger@famsik.de> | 2024-07-31 23:20:11 +0200 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2024-07-31 23:20:11 +0200 |
| commit | 52f7d33109973345b49b35e5289441e162a83c58 (patch) | |
| tree | 68d7ebc4aa82fbafa6e12b6829608a5827294a02 /compiler/rustc_pattern_analysis/src/errors.rs | |
| parent | 19f6ff065514d187a70ff917bbf6aebba4f87d28 (diff) | |
| parent | 91acacf85b9f81aeb41901f8918128bfe02946c8 (diff) | |
| download | rust-52f7d33109973345b49b35e5289441e162a83c58.tar.gz rust-52f7d33109973345b49b35e5289441e162a83c58.zip | |
Rollup merge of #128244 - compiler-errors:move-clone-sugg, r=estebank
Peel off explicit (or implicit) deref before suggesting clone on move error in borrowck, remove some hacks Also remove a heck of a lot of weird hacks in `suggest_cloning` that I don't think we should have around. I know this regresses tests, but I don't believe most of these suggestions were accurate, b/c: 1. They either produced type errors (e.g. turning `&x` into `x.clone()`) 2. They don't fix the issue 3. They fix the issue ostensibly, but introduce logic errors (e.g. cloning a `&mut Option<T>` to then `Option::take` out...) Most of the suggestions are still wrong, but they're not particularly *less* wrong IMO. Stacked on top of #128241, which is an "obviously worth landing" subset of this PR. r? estebank
Diffstat (limited to 'compiler/rustc_pattern_analysis/src/errors.rs')
0 files changed, 0 insertions, 0 deletions