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

Improve maybe_uninit_extra docs #86220

Merged
merged 1 commit into from
Jun 15, 2021
Merged

Conversation

est31
Copy link
Member

@est31 est31 commented Jun 11, 2021

For reasoning, see #63567 (comment)

@rust-highfive
Copy link
Collaborator

r? @scottmcm

(rust-highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jun 11, 2021
@rust-log-analyzer

This comment has been minimized.

@est31 est31 force-pushed the maybe-uninit-extra branch from f328e11 to 6036fdb Compare June 11, 2021 14:18
@rust-log-analyzer

This comment has been minimized.

@est31 est31 force-pushed the maybe-uninit-extra branch from 6036fdb to 24190f4 Compare June 11, 2021 14:50
@rust-log-analyzer

This comment has been minimized.

@RalfJung

This comment has been minimized.

@RalfJung
Copy link
Member

Argh, sorry, GH autocomplete went really wrong here...
r? @RalfJung

@rust-highfive rust-highfive assigned RalfJung and unassigned scottmcm Jun 13, 2021
@est31 est31 force-pushed the maybe-uninit-extra branch from e441af7 to ef6c9e6 Compare June 13, 2021 10:00
@est31 est31 force-pushed the maybe-uninit-extra branch from ef6c9e6 to 2375604 Compare June 13, 2021 19:00
@est31 est31 force-pushed the maybe-uninit-extra branch from 2375604 to db76996 Compare June 14, 2021 10:30
@RalfJung
Copy link
Member

r=me with the last wording nit resolved.
@bors delegate+

@bors
Copy link
Contributor

bors commented Jun 14, 2021

✌️ @est31 can now approve this pull request

@est31 est31 force-pushed the maybe-uninit-extra branch from db76996 to 8710258 Compare June 14, 2021 11:31
@est31
Copy link
Member Author

est31 commented Jun 14, 2021

@bors r=RalfJung rollup

@bors
Copy link
Contributor

bors commented Jun 14, 2021

📌 Commit 8710258 has been approved by RalfJung

@bors bors removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jun 14, 2021
@bors bors added the S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. label Jun 14, 2021
Dylan-DPC-zz pushed a commit to Dylan-DPC-zz/rust that referenced this pull request Jun 14, 2021
Dylan-DPC-zz pushed a commit to Dylan-DPC-zz/rust that referenced this pull request Jun 14, 2021
bors added a commit to rust-lang-ci/rust that referenced this pull request Jun 15, 2021
Rollup of 10 pull requests

Successful merges:

 - rust-lang#80269 (Explain non-dropped sender recv in docs)
 - rust-lang#82179 (Add functions `Duration::try_from_secs_{f32, f64}`)
 - rust-lang#85608 (Stabilize `ops::ControlFlow` (just the type))
 - rust-lang#85792 (Refactor windows sockets impl methods)
 - rust-lang#86220 (Improve maybe_uninit_extra docs)
 - rust-lang#86277 (Remove must_use from ALLOWED_ATTRIBUTES)
 - rust-lang#86285 (:arrow_up: rust-analyzer)
 - rust-lang#86294 (Stabilize {std, core}::prelude::rust_*.)
 - rust-lang#86306 (Add mailmap entries for myself)
 - rust-lang#86314 (Remove trailing triple backticks in `mut_keyword` docs)

Failed merges:

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit e84ee52 into rust-lang:master Jun 15, 2021
@rustbot rustbot added this to the 1.55.0 milestone Jun 15, 2021
JohnTitor added a commit to JohnTitor/rust that referenced this pull request Jul 12, 2021
Split MaybeUninit::write into new feature gate and stabilize it

This splits off the `MaybeUninit::write` function from the `maybe_uninit_extra` feature gate into a new `maybe_uninit_write` feature gate and stabilizes it.

Earlier work to improve the documentation of the write function: rust-lang#86220

Tracking issue: rust-lang#63567
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants