Skip to content

Remove RefCell::{with, with_mut} #13052

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

Merged
merged 2 commits into from
Mar 22, 2014
Merged

Conversation

sfackler
Copy link
Member

These are superfluous now that we have fixed rvalue lifetimes and Deref.

I'd also like to kill off get and set, but that'll be a large change so I want to make sure that we actually want to do that first.

These are superfluous now that we have fixed rvalue lifetimes and Deref.
@sfackler
Copy link
Member Author

@alexcrichton Updated with some marker type cleanup.

bors added a commit that referenced this pull request Mar 22, 2014
These are superfluous now that we have fixed rvalue lifetimes and Deref.

I'd also like to kill off `get` and `set`, but that'll be a large change so I want to make sure that we actually want to do that first.
@bors bors closed this Mar 22, 2014
@bors bors merged commit 1d98fe1 into rust-lang:master Mar 22, 2014
@sfackler sfackler deleted the clean-refcell branch May 15, 2014 05:02
flip1995 pushed a commit to flip1995/rust that referenced this pull request Jul 11, 2024
…Alexendoo

Lintcheck: Add `--warn-all` and make it the CI default

This PR adds a new `--warn-all` flag to lintcheck. This is intended for our CI, as it currently doesn't detect changes of `nursery` and `restriction` lints.

I only made it the default for the CI, as `restriction` lints tend to generate A LOT of lint triggers. Looking at you [`clippy::implicit_return`](https://rust-lang.github.io/rust-clippy/master/index.html#/clippy::implicit_return)

That's it. Should hopefully be easy to review.

Also, a bit thanks again to `@Alexendoo` for adding this to our CI ❤️

---

r? `@Alexendoo`

changelog: none
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.

4 participants