Skip to content

Rollup of 7 pull requests - #42336

Merged
bors merged 20 commits into
rust-lang:masterfrom
Mark-Simulacrum:rollup
May 31, 2017
Merged

Rollup of 7 pull requests#42336
bors merged 20 commits into
rust-lang:masterfrom
Mark-Simulacrum:rollup

Conversation

tommyipand others added 20 commits May 24, 2017 16:23
This makes the size_hint from things like `take` more precise.
Clarify docs on implementing Into.
This was suggested by @dtolnay in rust-lang#40380.
This explicitly clarifies in what circumstances you should implement `Into` instead of `From`.
…matsakis
Explain why a closure is `FnOnce` in closure errors.
Issue: rust-lang#42065@nikomatsakis Am I going the right direction with this?
~~I am stuck in a few bits:~~
~~1. How to trace the code to get the upvar instead of the original variable's span?~~
~~2. How to find the node id of the upvar where the move occured?~~
…komatsakis
Clarify the docs for align_of and its variants
It's okay to have unaligned raw pointers and then use `ptr::write_unaligned` and `ptr::read_unaligned`.
However, using unaligned `&T` and `&mut T` would be undefined behavior.
The current documentation seems to indicate that everything has to be aligned, but in reality only references do. This PR changes the text of docs accordingly.
r? @sfackler
…ta, r=nikomatsakis
Remove --crate-type=metadata deprecation warning
Fixesrust-lang#38640
…crichton
RangeFrom should have an infinite size_hint
Before,
```rust
(0..).take(4).size_hint() == (0, Some(4))
```
With this change,
```rust
(0..).take(4).size_hint() == (4, Some(4))
```
…chenkov
Parse macros named "default" correctly.
Fixesrust-lang#42292.
@rust-highfive

Copy link
Copy Markdown
Contributor

r? @eddyb

(rust_highfive has picked a reviewer for you, use r? to override)

@Mark-Simulacrum

Copy link
Copy Markdown
MemberAuthor

@bors r+ p=10

@bors

bors commented May 31, 2017

Copy link
Copy Markdown
Collaborator

📌 Commit 7f286a8 has been approved by Mark-Simulacrum

@bors

bors commented May 31, 2017

Copy link
Copy Markdown
Collaborator

⌛ Testing commit 7f286a8 with merge e0cc22b...

bors added a commit that referenced this pull request May 31, 2017
Rollup of 7 pull requests
- Successful merges: #42126, #42196, #42252, #42277, #42315, #42329, #42330
- Failed merges:
@bors

bors commented May 31, 2017

Copy link
Copy Markdown
Collaborator

☀️ Test successful - status-appveyor, status-travis
Approved by: Mark-Simulacrum
Pushing e0cc22b to master...

@CentrilCentril added the rollup A PR which is a rollup label Oct 2, 2019
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

rollupA PR which is a rollup

Projects

None yet

Development

Successfully merging this pull request may close these issues.

10 participants

@Mark-Simulacrum@rust-highfive@bors@eddyb@Centril@tommyip@citizen428@scottmcm@rap2hpoutre@qnighy