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

Don't trigger [debug_assert_with_mut_call] on debug_assert!(_.await) #5106

Merged
merged 4 commits into from
Jan 31, 2020

Conversation

flip1995
Copy link
Member

@flip1995 flip1995 commented Jan 29, 2020

Fixes #5105

cc #5112

changelog: Don't trigger [debug_assert_with_mut_call] on debug_assert!(_.await) and move it to nursery.

@flip1995 flip1995 added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Jan 29, 2020
@flip1995 flip1995 force-pushed the dbg_assert_mut_async branch 2 times, most recently from 9979ab7 to 1cc62de Compare January 30, 2020 14:23
@flip1995 flip1995 force-pushed the dbg_assert_mut_async branch from 1cc62de to 7b302e9 Compare January 31, 2020 09:46
@flip1995 flip1995 force-pushed the dbg_assert_mut_async branch from 7b302e9 to 5ba4aa8 Compare January 31, 2020 09:57
@flip1995
Copy link
Member Author

@rust-lang/clippy We should merge this ASAP (at least the move to nursery) and probably backport it to beta. #5112

@oli-obk
Copy link
Contributor

oli-obk commented Jan 31, 2020

@bors r+

@bors
Copy link
Contributor

bors commented Jan 31, 2020

📌 Commit 5ba4aa8 has been approved by oli-obk

@bors
Copy link
Contributor

bors commented Jan 31, 2020

⌛ Testing commit 5ba4aa8 with merge 5826a04...

bors added a commit that referenced this pull request Jan 31, 2020
Don't trigger [debug_assert_with_mut_call] on debug_assert!(_.await)

Fixes #5105

cc #5112

changelog: Don't trigger [`debug_assert_with_mut_call`] on `debug_assert!(_.await)` and move it to nursery.
@bors
Copy link
Contributor

bors commented Jan 31, 2020

☀️ Test successful - checks-travis, status-appveyor
Approved by: oli-obk
Pushing 5826a04 to master...

@bors bors merged commit 5ba4aa8 into rust-lang:master Jan 31, 2020
@flip1995 flip1995 deleted the dbg_assert_mut_async branch January 31, 2020 18:37
@flip1995
Copy link
Member Author

Thanks! I'll do the backport tomorrow.

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.

Error when using debug_assert! and async function
3 participants