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

Rollup of 15 pull requests #67670

Merged
merged 33 commits into from
Dec 28, 2019
Merged

Rollup of 15 pull requests #67670

merged 33 commits into from
Dec 28, 2019

Conversation

oli-obk
Copy link
Contributor

@oli-obk oli-obk commented Dec 27, 2019

Successful merges:

Failed merges:

r? @ghost

king6cong and others added 30 commits December 24, 2019 12:44
`ptr` of `slice::Iter` and `slice::IterMut` can never be null, but this
fact wasn't exploited for layout optimizations. By changing `ptr` from
`*<mutability> T` to `NonNull<T>`, the compiler can now optimize layout
of `Option<Iter<'a, T>>`.
add IntoFuture trait and support for await

The [async-await RFC](https://rust-lang.github.io/rfcs/2394-async_await.html#the-await-compiler-built-in) mentions being able to `await` anything implementing `IntoFuture`. Somewhere along the way, it was left out.
reuse `capacity` variable in slice::repeat

None
Use NonNull in slice::Iter and slice::IterMut.

`ptr` of `slice::Iter` and `slice::IterMut` can never be null, but this
fact wasn't exploited for layout optimizations. By changing `ptr` from
`*<mutability> T` to `NonNull<T>`, the compiler can now optimize layout
of `Option<Iter<'a, T>>`.

Closes rust-lang#67228
…-intrinsics, r=Centril

Use issue = "none" instead of "0" in intrinsics

`issue = "0"` was incorrectly used in rust-lang@9ee16e1, the project is moving away from issue 0
…docs, r=Dylan-DPC

Remove `compiler_builtins_lib` documentation

Fixes rust-lang#67593
…r=oli-obk

Use the correct type for static qualifs

Closes rust-lang#67609
…r=steveklabnik

Remove redundant link texts

Most of these links are followed by a parenthesized expression.  I think that the redundant link texts were added to prevent interpretation as an inline link.  This is unnecessary since the closing square bracket and opening parenthesis are separated by whitespace.
…nks, r=steveklabnik

Convert collapsed to shortcut reference links
…dtolnay

Document safety of Path casting

I would personally feel more comfortable making the relevant (internal anyway) types repr(transparent) and then documenting that we can make these casts because of that, but I believe this is a more minimal PR, so posting it first.

Resolves rust-lang#45910.
…ion-test, r=Centril

Add regression test for old NLL ICE

This fails on nightly-2018-06-24.

Resolves rust-lang#51770
@oli-obk
Copy link
Contributor Author

oli-obk commented Dec 27, 2019

@bors r+ p=15

@bors
Copy link
Contributor

bors commented Dec 27, 2019

📌 Commit 65bbcf0 has been approved by oli-obk

@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 Dec 27, 2019
@bors
Copy link
Contributor

bors commented Dec 28, 2019

⌛ Testing commit 65bbcf0 with merge 3a087ad...

bors added a commit that referenced this pull request Dec 28, 2019
Rollup of 15 pull requests

Successful merges:

 - #65244 (add IntoFuture trait and support for await)
 - #67576 (reuse `capacity` variable in slice::repeat)
 - #67588 (Use NonNull in slice::Iter and slice::IterMut.)
 - #67594 (Update libc to 0.2.66)
 - #67602 (Use issue = "none" instead of "0" in intrinsics)
 - #67604 (Add Scalar::to_(u|i)16 methods)
 - #67617 (Remove `compiler_builtins_lib` documentation)
 - #67621 (Use the correct type for static qualifs)
 - #67629 (Remove redundant link texts)
 - #67632 (Convert collapsed to shortcut reference links)
 - #67633 (Update .mailmap)
 - #67635 (Document safety of Path casting)
 - #67654 (Add regression test for old NLL ICE)
 - #67659 (Stabilize the `matches!` macro)
 - #67664 (Fix some mailmap entries)

Failed merges:

r? @ghost
@JohnTitor JohnTitor added the rollup A PR which is a rollup label Dec 28, 2019
@bors
Copy link
Contributor

bors commented Dec 28, 2019

☀️ Test successful - checks-azure
Approved by: oli-obk
Pushing 3a087ad to master...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merged-by-bors This PR was explicitly merged by bors. rollup A PR which is a rollup 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.