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

Propagate mutability from arguments to local bindings in async fn #60501

Merged
merged 1 commit into from
May 3, 2019

Conversation

taiki-e
Copy link
Member

@taiki-e taiki-e commented May 3, 2019

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label May 3, 2019
@cramertj
Copy link
Member

cramertj commented May 3, 2019

@bors r+

@bors
Copy link
Contributor

bors commented May 3, 2019

📌 Commit 2fe50bc has been approved by cramertj

@bors bors 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 May 3, 2019
@Centril
Copy link
Contributor

Centril commented May 3, 2019

Could you please extend the test suite to also test for ref and ref mut patterns as well as those + mut in nested positions?

EDIT: made a E-needstest issue for this.

Centril added a commit to Centril/rust that referenced this pull request May 3, 2019
…ts, r=cramertj

Propagate mutability from arguments to local bindings in async fn

Fixes rust-lang#60498

cc @nikomatsakis
r? @davidtwco
bors added a commit that referenced this pull request May 3, 2019
Rollup of 12 pull requests

Successful merges:

 - #59928 (Make deprecation lint `ambiguous_associated_items` deny-by-default)
 - #60220 (report fatal errors during doctest parsing)
 - #60373 (Tidy: ensure lang features are sorted by since)
 - #60388 (Disallow non-explicit elided lifetimes in async fn)
 - #60393 ( Do not suggest incorrect syntax on pattern type error due to borrow)
 - #60401 (Rename `RUST_LOG` to `RUSTC_LOG`)
 - #60409 (Require a trait in the bounds of existential types)
 - #60455 (Resolve match arm ty when arms diverge)
 - #60457 (Const prop refactoring)
 - #60467 (Avoid repeated interning of static strings.)
 - #60478 (minor compiler doc tweaks)
 - #60501 (Propagate mutability from arguments to local bindings in async fn)

Failed merges:

r? @ghost
@bors bors merged commit 2fe50bc into rust-lang:master May 3, 2019
@taiki-e taiki-e deleted the async-await-mutable-arguments branch May 3, 2019 18:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

async fn forgets the mutability of the arguments
6 participants