Skip to content

Add coercions from *mut to *const and from &mut to *const. #19766

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
Dec 17, 2014

Conversation

nrc
Copy link
Member

@nrc nrc commented Dec 12, 2014

r?

@nrc
Copy link
Member Author

nrc commented Dec 12, 2014

part of #18469

@Gankra
Copy link
Contributor

Gankra commented Dec 12, 2014

🎊 🎊 🎉

🔥 as *const 🔥 😈

@jdm
Copy link
Contributor

jdm commented Dec 12, 2014

These are tears of joy on my face.

bors added a commit that referenced this pull request Dec 14, 2014
Add coercions from *mut to *const and from &mut to *const.

Reviewed-by: eddyb
bors referenced this pull request Dec 16, 2014
This creates an enormous amount of spew.
bors added a commit that referenced this pull request Dec 16, 2014
Add coercions from *mut to *const and from &mut to *const.

Reviewed-by: eddyb
bors referenced this pull request Dec 16, 2014
Using a type alias for iterator implementations is fragile since this
exposes the implementation to users of the iterator, and any changes
could break existing code.

This commit changes the iterators of `VecMap` to use
proper new types, rather than type aliases.  However, since it is
fair-game to treat a type-alias as the aliased type, this is a:

[breaking-change].
bors added a commit that referenced this pull request Dec 16, 2014
Add coercions from *mut to *const and from &mut to *const.

Reviewed-by: eddyb
brson added a commit to brson/rust that referenced this pull request Dec 16, 2014
bors added a commit that referenced this pull request Dec 17, 2014
Add coercions from *mut to *const and from &mut to *const.

Reviewed-by: eddyb
alexcrichton added a commit to alexcrichton/rust that referenced this pull request Dec 17, 2014
@bors bors merged commit 19eb4bf into rust-lang:master Dec 17, 2014
alexcrichton added a commit to alexcrichton/rust that referenced this pull request Jan 19, 2015
After PR rust-lang#19766 added implicit coersions `*mut T -> *const T`, the explicit casts can be removed.
(The number of such casts turned out to be relatively small).
barosl added a commit to barosl/rust that referenced this pull request Jan 20, 2015
After PR rust-lang#19766 added implicit coersions `*mut T -> *const T`, the explicit casts can be removed.
(The number of such casts turned out to be relatively small).
# 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.

6 participants