Skip to content

Deprecated if_let_mutex due to stabilization in 2024 edition #13686

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

Closed
wants to merge 1 commit into from

Conversation

alex-semenyuk
Copy link
Member

Close #13679

As mentioned at #131154 if_let_rescope is now stable for Edition 2024, so no need if_let_mutex

changelog: [if_let_mutex]: deprecation

@rustbot
Copy link
Collaborator

rustbot commented Nov 14, 2024

r? @Alexendoo

rustbot has assigned @Alexendoo.
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

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Nov 14, 2024
@alex-semenyuk alex-semenyuk marked this pull request as draft November 14, 2024 17:13
@xFrednet
Copy link
Member

Does the stabilization effect all editions or only Edition 2024 and onwards?

@alex-semenyuk
Copy link
Member Author

Does the stabilization effect all editions or only Edition 2024 and onwards?

Suppose only 2024

@xFrednet
Copy link
Member

I haven't looked at the lint in detail, but would it make sense to make the linting dependent on the edition instead of deprecating it?

@alex-semenyuk
Copy link
Member Author

Make sense, might you know examples such edition dependent issues

@y21
Copy link
Member

y21 commented Nov 14, 2024

Make sense, might you know examples such edition dependent issues

manual_c_str_literals is one that only lints in Edition >= 2021 crates

&& cx.tcx.sess.edition() >= Edition2021

@alex-semenyuk
Copy link
Member Author

When I try to use //@edition:2024 something like here
it gives
error: edition 2024 is unstable and only available with -Z unstable-options

@est31
Copy link
Member

est31 commented Nov 16, 2024

One can work around that issue by passing the option manually. I don't know how that test doesn't trip the same issue. Maybe CI specifies RUSTFLAGS and adds -Zunstable-options for some other reason?

//@ compile-flags: -Zunstable-options

@Alexendoo
Copy link
Member

The linked test is fine since it's only editions 2018 and 2021

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.

Should disable if_let_mutex in 2024 edition
6 participants