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

tutorial: alternatives to ownership #10589

Merged
merged 1 commit into from
Nov 22, 2013
Merged

tutorial: alternatives to ownership #10589

merged 1 commit into from
Nov 22, 2013

Conversation

thestinger
Copy link
Contributor

This replaces the old section on managed pointers because the syntax is
going to be removed and it's currently feature gated so the examples
don't work out-of-the-box. Dynamic mutability coverage can be added
after the Mut<T> work has landed.

This replaces the old section on managed pointers because the syntax is
going to be removed and it's currently feature gated so the examples
don't work out-of-the-box. Dynamic mutability coverage can be added
after the `Mut<T>` work has landed.
bors added a commit that referenced this pull request Nov 22, 2013

This replaces the old section on managed pointers because the syntax is
going to be removed and it's currently feature gated so the examples
don't work out-of-the-box. Dynamic mutability coverage can be added
after the `Mut<T>` work has landed.
@bors bors closed this Nov 22, 2013
@bors bors merged commit c06ce4c into rust-lang:master Nov 22, 2013
flip1995 pushed a commit to flip1995/rust that referenced this pull request Apr 6, 2023
Mini-fix `double_must_use` for async functions

From Rust 1.67 onwards, the `#[must_use]` attribute also applies to the `Future::Output` (rust-lang#100633). So the lint `double_must_use` was linting all async functions. This PR changes the `double_must_use` lint so it ignores `async` functions.

---

Closes rust-lang#10486
changelog: [`double_must_use`]: Fix false positive in async function
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants