Skip to content

rustc: remove some unnecessary transmutes. #12394

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 1 commit into from
Feb 20, 2014
Merged

Conversation

huonw
Copy link
Member

@huonw huonw commented Feb 19, 2014

These can all be done by implicit or explicit &T -> *T casts, which are
more restricted and so are safer.

These can all be done by implicit or explicit &T -> *T casts, which are
more restricted and so are safer.
@flaper87
Copy link
Contributor

LGTM

@alexcrichton
Copy link
Member

Rust just got ~1% safer! (4/441 transmutes removed)

bors added a commit that referenced this pull request Feb 20, 2014
These can all be done by implicit or explicit &T -> *T casts, which are
more restricted and so are safer.
@bors bors closed this Feb 20, 2014
@bors bors merged commit dcee327 into rust-lang:master Feb 20, 2014
@huonw huonw deleted the transmute-- branch June 27, 2014 06:48
JohnTitor pushed a commit to JohnTitor/rust that referenced this pull request Aug 30, 2022
…ng, r=Veykril

feat: make trait assoc items become inactive due to cfg

fixes rust-lang#12394
flip1995 pushed a commit to flip1995/rust that referenced this pull request Mar 7, 2024
…ndoo

Refactor lints in clippy_lints::attrs into separate submodules/files

This pull request contains the changes requested in issue rust-lang#12390.

changelog: none
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants