Skip to content

Fix the issue of invalid suggestion for a reference of iterator - #127669

Merged
bors merged 2 commits into
rust-lang:masterfrom
chenyukang:yukang-fix-deref-127590
Jul 16, 2024
Merged

Fix the issue of invalid suggestion for a reference of iterator#127669
bors merged 2 commits into
rust-lang:masterfrom
chenyukang:yukang-fix-deref-127590

Conversation

@chenyukang

Copy link
Copy Markdown
Member

Fixes#127590

@rustbot

Copy link
Copy Markdown
Collaborator

r? @nnethercote

rustbot has assigned @nnethercote.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

@rustbotrustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Jul 12, 2024
LL + for (src, dest) in std::iter::zip(fields.iter(), variant.iter().clone()) {
|

error[E0277]: `&std::slice::Iter<'_, {integer}>` is not an iterator

Copy link
Copy Markdown
MemberAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this diag also seems weird, it's better if we could remove it.

@nnethercote

Copy link
Copy Markdown
Contributor

Could you break this into two commits?

  • First commit adds the new test, with the old test output.
  • Second commit makes the change, and updates the test output.

This makes it much easier to review and see the effect of the change. Thanks!

@chenyukang
chenyukangforce-pushed the yukang-fix-deref-127590 branch from c8ed5c5 to 5ebfe69CompareJuly 16, 2024 14:00
@chenyukang
chenyukangforce-pushed the yukang-fix-deref-127590 branch from 5ebfe69 to 48ddf5eCompareJuly 16, 2024 14:02
@nnethercote

Copy link
Copy Markdown
Contributor

Great, thanks!

@bors r+ rollup

@bors

bors commented Jul 16, 2024

Copy link
Copy Markdown
Collaborator

📌 Commit 48ddf5e has been approved by nnethercote

It is now in the queue for this repository.

@borsbors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Jul 16, 2024
bors added a commit to rust-lang-ci/rust that referenced this pull request Jul 16, 2024
…iaskrgr
Rollup of 8 pull requests
Successful merges:
- rust-lang#127669 (Fix the issue of invalid suggestion for a reference of iterator)
- rust-lang#127707 (match lowering: Use an iterator to find `expand_until`)
- rust-lang#127730 (Fix and enforce `unsafe_op_in_unsafe_fn` in compiler)
- rust-lang#127789 (delete #![allow(unsafe_op_in_unsafe_fn)] in teeos)
- rust-lang#127805 (run-make-support: update gimli to 0.31.0)
- rust-lang#127808 (Make ErrorGuaranteed discoverable outside types, consts, and lifetimes)
- rust-lang#127817 (Fix a bunch of sites that were walking instead of visiting, making it impossible for visitor impls to look at these values)
- rust-lang#127818 (Various ast validation simplifications)
r? `@ghost`
`@rustbot` modify labels: rollup
@bors
bors merged commit 5ee4533 into rust-lang:masterJul 16, 2024
@rustbotrustbot added this to the 1.81.0 milestone Jul 16, 2024
rust-timer added a commit to rust-lang-ci/rust that referenced this pull request Jul 16, 2024
Rollup merge of rust-lang#127669 - chenyukang:yukang-fix-deref-127590, r=nnethercote
Fix the issue of invalid suggestion for a reference of iterator
Fixesrust-lang#127590
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-borsStatus: Waiting on bors to run and complete tests. Bors will change the label on completion.T-compilerRelevant to the compiler team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Invalid suggestion for a reference of iterator

4 participants

@chenyukang@rustbot@nnethercote@bors