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

Import #[repr(transparent)] docs from the Unstable book #353

Closed
wants to merge 1 commit into from

Conversation

SimonSapin
Copy link

… as the feature is ready for stabilization: rust-lang/rust#43036 (comment)

@hanna-kruppe
Copy link

I don't think this text really fits into the reference, as it explains a lot of motivation and background rather than precisely and concisely stating what the attribute does. Certainly when I wrote it I had something like The Book or RBE in mind, not the reference.

@SimonSapin
Copy link
Author

Feature stabilization PR: rust-lang/rust#51395

I agree that the subsections do not necessarily belong there as-is but I wasn’t sure what else to do, this PR is sort of a straw man.

SimonSapin added a commit to SimonSapin/rust that referenced this pull request Jun 12, 2018
bors added a commit to rust-lang/rust that referenced this pull request Jun 16, 2018
Stabilize #[repr(transparent)]

Tracking issue FCP: #43036 (comment)
Reference PR: rust-lang/reference#353
@alercah
Copy link
Contributor

alercah commented Aug 3, 2018

I agree that this is not ready for integration into the reference. Adding more/confusing text is not necessarily better, even if it improves coverage.

@Havvy
Copy link
Contributor

Havvy commented Aug 20, 2018

Closing in favor of #389 since the RFC's contents don't translate to being in the reference directly that well.

@Havvy Havvy closed this Aug 20, 2018
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.

4 participants