-
Notifications
You must be signed in to change notification settings - Fork 12.3k
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
rustc: remove some unnecessary
transmute
s.
These can all be done by implicit or explicit &T -> *T casts, which are more restricted and so are safer.
- Loading branch information
Showing
3 changed files
with
4 additions
and
12 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
dcee327
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
saw approval from pnkfelix
at huonw@dcee327
dcee327
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
merging huonw/rust/transmute-- = dcee327 into auto
dcee327
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
huonw/rust/transmute-- = dcee327 merged ok, testing candidate = 801f8f6
dcee327
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
all tests pass:
success: http://buildbot.rust-lang.org/builders/auto-mac-32-opt/builds/4348
success: http://buildbot.rust-lang.org/builders/auto-mac-32-nopt-c/builds/2190
success: http://buildbot.rust-lang.org/builders/auto-mac-32-nopt-t/builds/2191
success: http://buildbot.rust-lang.org/builders/auto-mac-64-opt/builds/4346
success: http://buildbot.rust-lang.org/builders/auto-mac-64-nopt-c/builds/3446
success: http://buildbot.rust-lang.org/builders/auto-mac-64-nopt-t/builds/3452
success: http://buildbot.rust-lang.org/builders/auto-linux-32-opt/builds/4361
success: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-c/builds/3449
success: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-t/builds/3455
success: http://buildbot.rust-lang.org/builders/auto-linux-64-opt/builds/4363
success: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-c/builds/3449
success: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-t/builds/3454
success: http://buildbot.rust-lang.org/builders/auto-linux-64-x-android/builds/3523
success: http://buildbot.rust-lang.org/builders/auto-linux-64-x-android-t/builds/1302
success: http://buildbot.rust-lang.org/builders/auto-win-32-opt/builds/4355
success: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-c/builds/3456
success: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-t/builds/3462
success: http://buildbot.rust-lang.org/builders/auto-bsd-64-opt/builds/4123
dcee327
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fast-forwarding master to auto = 801f8f6