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 10 pull requests #32860

Closed
wants to merge 18 commits into from
Closed

Conversation

@Manishearth
Copy link
Member Author

@bors r+ p=20

@bors
Copy link
Contributor

bors commented Apr 9, 2016

📌 Commit a7246af has been approved by Manishearth

@rust-highfive
Copy link
Collaborator

r? @alexcrichton

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

@bors
Copy link
Contributor

bors commented Apr 10, 2016

⌛ Testing commit a7246af with merge ddb6667...

@bors
Copy link
Contributor

bors commented Apr 10, 2016

💔 Test failed - auto-linux-32-opt

@Manishearth
Copy link
Member Author

@bors r+ force

@bors
Copy link
Contributor

bors commented Apr 10, 2016

📌 Commit 891e2ff has been approved by Manishearth

@Manishearth
Copy link
Member Author

@bors force

1 similar comment
@Manishearth
Copy link
Member Author

@bors force

@bors
Copy link
Contributor

bors commented Apr 10, 2016

⌛ Testing commit 891e2ff with merge 5c754a6...

@bors
Copy link
Contributor

bors commented Apr 10, 2016

💔 Test failed - auto-win-msvc-32-opt

@Manishearth
Copy link
Member Author

@bors r+ force

@bors
Copy link
Contributor

bors commented Apr 10, 2016

📌 Commit 220744b has been approved by Manishearth

@Manishearth
Copy link
Member Author

@bors force

@bors
Copy link
Contributor

bors commented Apr 10, 2016

⌛ Testing commit 220744b with merge 0f62b61...

@bors
Copy link
Contributor

bors commented Apr 10, 2016

💔 Test failed - auto-win-msvc-32-opt

Adds data race in docs

Thanks for all your hard work!
This is in reference to rust-lang#32733
I know there has been a discussion about this on PR rust-lang#32538 so you are welcome to keep the code as is or merge my documentation in.
Let me know what you think and/or if you want me to modify anything!
Blacklist fn item types from being used with variadic functions.

Fixes rust-lang#32201 by adding fn types to the variadic blacklist which currently includes `bool`, `i8`, `u8`, `i16`, `u16` and `f32`.
…tics, r=eddyb

resolve: import resolution diagnostics

This improves the diagnostics for failing import resolutions (fixes rust-lang#32833).
r? @eddyb
resolve: Add regression tests for fixed issues

This adds regression tests for fixed issues in resolve (closes rust-lang#22146, closes rust-lang#24883, closes rust-lang#26930).
r? @eddyb
…chton

Don't read past limit for in BufRead instance of Take

Similar to `Read::read`, `BufRead::fill_buf` impl of `Take` should not call `inner.fill_buf` if the limit is already reached.
@Manishearth
Copy link
Member Author

@bors r+

@bors
Copy link
Contributor

bors commented Apr 10, 2016

📌 Commit db8f4d6 has been approved by Manishearth

@bors
Copy link
Contributor

bors commented Apr 10, 2016

⌛ Testing commit db8f4d6 with merge 43a6e22...

@bors
Copy link
Contributor

bors commented Apr 10, 2016

💔 Test failed - auto-win-msvc-32-opt

@Centril Centril added the rollup A PR which is a rollup label Oct 24, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
rollup A PR which is a rollup
Projects
None yet
Development

Successfully merging this pull request may close these issues.