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

Start leveraging the Deref/DerefMut traits #13053

Closed

Conversation

alexcrichton
Copy link
Member

This commit removes the get() method from Ref and RefMut in favor of the * operator, and removes all usage of the deref() function manually from rustc, favoring using * instead.

Some of the code is a little wacky, but that's due to either #13044 or #13042

@huonw
Copy link
Member

huonw commented Mar 22, 2014

(Unsurprisingly needs a rebase.)

bors added a commit that referenced this pull request Mar 22, 2014
This commit removes the `get()` method from `Ref` and `RefMut` in favor of the `*` operator, and removes all usage of the `deref()` function manually from rustc, favoring using `*` instead.

Some of the code is a little wacky, but that's due to either #13044 or #13042
@bors bors closed this Mar 22, 2014
@alexcrichton alexcrichton deleted the removing-ref-cell-get branch March 22, 2014 18:11
lnicola pushed a commit to lnicola/rust that referenced this pull request Aug 23, 2022
fix: resolve path `Self` alone in value namespace

Fixes rust-lang#12968
dingxiangfei2009 pushed a commit to dingxiangfei2009/rust that referenced this pull request Jul 28, 2024
… r=llogiq

Add `BTreeSet` detection to the `set_contains_or_insert` lint

* Detect `BTreeSet::contains` + `BTreeSet::insert` usage in the same way as with the `HashSet`.
CC: `@lochetti` `@bitfield`

----

changelog: [`set_contains_or_insert`]: Handle `BTreeSet` in addition to `HashSet`
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.

3 participants