Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[unused_enumerate_index]: don't ICE on empty tuples #11756

Merged
merged 1 commit into from
Nov 3, 2023

Conversation

y21
Copy link
Member

@y21 y21 commented Nov 3, 2023

Fixes #11755

changelog: [unused_enumerate_index]: don't ICE on empty tuples

I'm going to nominate for beta backport because the code that is needed to trigger this seems likely to occur in real code
@rustbot label +beta-nominated

@rustbot
Copy link
Collaborator

rustbot commented Nov 3, 2023

r? @Manishearth

(rustbot has picked a reviewer for you, use r? to override)

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties beta-nominated Nominated for backporting to the compiler in the beta channel. labels Nov 3, 2023
@matthiaskrgr
Copy link
Member

I'm not sure if this affects beta, looks like this only popped up right after the sync that merged today.

@y21
Copy link
Member Author

y21 commented Nov 3, 2023

oh right
@rustbot label -beta-nominated

@rustbot rustbot removed the beta-nominated Nominated for backporting to the compiler in the beta channel. label Nov 3, 2023
@matthiaskrgr
Copy link
Member

yeah, no beta ICE, we might still want to backport the fix directly into rust-lang/rust master though :)

@Manishearth
Copy link
Member

@bors r+

@bors
Copy link
Contributor

bors commented Nov 3, 2023

📌 Commit 294df80 has been approved by Manishearth

It is now in the queue for this repository.

@Manishearth
Copy link
Member

Yeah let's backport into rust-lang/rust

@bors
Copy link
Contributor

bors commented Nov 3, 2023

⌛ Testing commit 294df80 with merge 789bc73...

@bors
Copy link
Contributor

bors commented Nov 3, 2023

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: Manishearth
Pushing 789bc73 to master...

@bors bors merged commit 789bc73 into rust-lang:master Nov 3, 2023
5 checks passed
@matthiaskrgr matthiaskrgr added the beta-nominated Nominated for backporting to the compiler in the beta channel. label Nov 12, 2023
bors added a commit to rust-lang-ci/rust that referenced this pull request Dec 21, 2023
…troalbini

[beta] Clippy beta backport

PR towards stable, as beta was branched a day early and I missed the notification.

- rust-lang/rust-clippy#11538
- rust-lang/rust-clippy#11756
- rust-lang/rust-clippy#11760
- rust-lang/rust-clippy#11953

r? `@pietroalbini`
bors added a commit to rust-lang-ci/rust that referenced this pull request Dec 21, 2023
…troalbini

[beta] Clippy beta backport

PR towards stable, as beta was branched a day early and I missed the notification.

- rust-lang/rust-clippy#11538
- rust-lang/rust-clippy#11756
- rust-lang/rust-clippy#11760
- rust-lang/rust-clippy#11953

r? `@pietroalbini`
@flip1995 flip1995 added beta-accepted Accepted for backporting to the compiler in the beta channel. and removed beta-nominated Nominated for backporting to the compiler in the beta channel. labels Jan 4, 2024
@xFrednet xFrednet removed the beta-accepted Accepted for backporting to the compiler in the beta channel. label Mar 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties
Projects
None yet
Development

Successfully merging this pull request may close these issues.

unused_enumerate_index.rs: index out of bounds
7 participants