Skip to content
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

Add union items to the reference #57

Merged
merged 1 commit into from
May 30, 2017

Conversation

petrochenkov
Copy link
Contributor

Copy-pasted from rust-lang/rfcs#1897

@steveklabnik
Copy link
Member

Unions are not yet stable, so we'll have to wait until they are to merge.

@petrochenkov
Copy link
Contributor Author

@steveklabnik
So, the stabilization PR (rust-lang/rust#42068) can land before this one? Ok.

@steveklabnik
Copy link
Member

Yup! Sending in this PR is enough to satisfy the requirement of stabilizing the feature. Then it lands, then we land tihs.

bors added a commit to rust-lang/rust that referenced this pull request May 27, 2017
Stabilize unions with `Copy` fields and no destructor

What else is needed:
- [x] Documentation (rust-lang/reference#57).
- [x] Making assignments to `Copy` union fields safe (#42083).
- [ ] Backport? (The "stabilization decision" is from [Apr 13](#32836 (comment)), it's just this PR is late.)

cc #32836
r? @nikomatsakis
@frewsxcv
Copy link
Member

Just reran Travis and this passes now. The contents of the PR look good to me. Seems good to merge?

@steveklabnik steveklabnik merged commit e3479cb into rust-lang:master May 30, 2017
@steveklabnik
Copy link
Member

Yup!

@petrochenkov petrochenkov deleted the union branch September 10, 2017 21:54
# 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.

3 participants