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

Tracking issue for ptr::{read,write}_unaligned #37955

Closed
alexcrichton opened this Issue Nov 23, 2016 · 12 comments

Comments

Projects
None yet
6 participants
@alexcrichton
Copy link
Member

alexcrichton commented Nov 23, 2016

Tracking issue for rust-lang/rfcs#1725

@sfackler sfackler added B-unstable and removed B-RFC-approved labels Nov 24, 2016

@sfackler sfackler added B-RFC-approved and removed B-unstable labels Dec 11, 2016

sfackler added a commit to sfackler/rust that referenced this issue Dec 12, 2016

sfackler added a commit to sfackler/rust that referenced this issue Dec 12, 2016

sfackler added a commit to sfackler/rust that referenced this issue Dec 12, 2016

sfackler added a commit to sfackler/rust that referenced this issue Dec 12, 2016

sfackler added a commit to sfackler/rust that referenced this issue Dec 13, 2016

bors added a commit that referenced this issue Dec 13, 2016

Auto merge of #38309 - sfackler:rfc-1725, r=alexcrichton
Implement RFC #1725 (read_unaligned, write_unaligned)

cc #37955

r? @alexcrichton
@chriskrycho

This comment has been minimized.

Copy link
Contributor

chriskrycho commented Dec 30, 2016

Status check: it looks like this is implemented, behind a feature flag, and not-yet-documented?

@whitequark

This comment has been minimized.

Copy link
Member

whitequark commented Feb 8, 2017

This has documentation now. What's the requirement for stabilization?

@alexcrichton

This comment has been minimized.

Copy link
Member Author

alexcrichton commented Feb 8, 2017

Ah looks like this wasn't tagged B-unstable by accident which then fell under our radar. In that case...

@rfcbot fcp merge

@rfcbot

This comment has been minimized.

Copy link

rfcbot commented Feb 8, 2017

Team member @alexcrichton has proposed to merge this. The next step is review by the rest of the tagged teams:

No concerns currently listed.

Once these reviewers reach consensus, this will enter its final comment period. If you spot a major issue that hasn't been raised at any point in this process, please speak up!

See this document for info about what commands tagged team members can give me.

@niconii

This comment has been minimized.

Copy link
Member

niconii commented Feb 11, 2017

The problem with ptr::write's docs mentioned in #39733 is true of ptr::write_unaligned's docs as well.

@rfcbot

This comment has been minimized.

Copy link

rfcbot commented Feb 13, 2017

🔔 This is now entering its final comment period, as per the review above. 🔔

4 similar comments
@rfcbot

This comment has been minimized.

Copy link

rfcbot commented Feb 13, 2017

🔔 This is now entering its final comment period, as per the review above. 🔔

@rfcbot

This comment has been minimized.

Copy link

rfcbot commented Feb 13, 2017

🔔 This is now entering its final comment period, as per the review above. 🔔

@rfcbot

This comment has been minimized.

Copy link

rfcbot commented Feb 13, 2017

🔔 This is now entering its final comment period, as per the review above. 🔔

@rfcbot

This comment has been minimized.

Copy link

rfcbot commented Feb 13, 2017

🔔 This is now entering its final comment period, as per the review above. 🔔

@whitequark

This comment has been minimized.

Copy link
Member

whitequark commented Feb 13, 2017

@rfcbot calm down please

@rfcbot

This comment has been minimized.

Copy link

rfcbot commented Feb 23, 2017

The final comment period is now complete.

frewsxcv pushed a commit to frewsxcv/rust that referenced this issue Mar 17, 2017

alexcrichton added a commit to alexcrichton/rust that referenced this issue Mar 17, 2017

alexcrichton added a commit to alexcrichton/rust that referenced this issue Mar 17, 2017

@bors bors closed this in d38ea8b Mar 19, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.