Skip to content

core: Rename to_mut and from_mut to cast_to_mut and cast_from_mut #4596

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 1 commit into from
Jan 23, 2013
Merged

core: Rename to_mut and from_mut to cast_to_mut and cast_from_mut #4596

merged 1 commit into from
Jan 23, 2013

Conversation

Trinick
Copy link

@Trinick Trinick commented Jan 23, 2013

Solution to issue #3355 asking for the renaming of vec::to_mut/from_mut to cast_to_mut/cast_from_mut

catamorphism added a commit that referenced this pull request Jan 23, 2013
core: Rename to_mut and from_mut to cast_to_mut and cast_from_mut
@catamorphism catamorphism merged commit 995ccc0 into rust-lang:incoming Jan 23, 2013
@catamorphism
Copy link
Contributor

Thanks for the patch! In the future, please run make check before submitting a pull request, as some of the compiler unit tests (which don't get compiled when you just build the compiler) were not updated in the new names. I'll fix it this time, though.

@Trinick
Copy link
Author

Trinick commented Jan 23, 2013

I thought I patched all the tests, I did run make check right before sending the pull request. Sorry if I missed some. Thanks!

bors pushed a commit to rust-lang-ci/rust that referenced this pull request Oct 24, 2020
This is based almost entirely on the code available in the previous PR rust-lang#4596.
bors added a commit to rust-lang-ci/rust that referenced this pull request Oct 24, 2020
New lint: Recommend using `ptr::eq` when possible

This is based almost entirely on the code available in the previous PR rust-lang#4596. I merely updated the code to make it compile.

Fixes rust-lang#3661.

- [ ] I'm not sure about the lint name, but it was the one used in the original PR.
- [X] Added passing UI tests (including committed `.stderr` file)
- [X] `cargo test` passes locally
- [X] Executed `cargo dev update_lints`
- [X] Added lint documentation
- [X] Run `cargo dev fmt`

---

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.

2 participants