Skip to content

Rollup of 9 pull requests - #41533

Closed
arielb1 wants to merge 22 commits into
rust-lang:masterfrom
arielb1:rollup
Closed

Rollup of 9 pull requests#41533
arielb1 wants to merge 22 commits into
rust-lang:masterfrom
arielb1:rollup

Conversation

petrochenkovand others added 22 commits April 16, 2017 21:19
Also spell out that read and write operations should be retried on
`ErrorKind::Interrupted` errors.
Fixesrust-lang#38494.
Turns out all six of these impls are incorrect.
Adds rust-windbg.cmd script
Adds rust-gdb/rust-lldb equivalent for windbg that loads the Rust .natvis files on start.
This change modifies the bootstrap code to add rust-windbg to bin and the .natvis files to lib/rustlib/etc.
Example usage from cmd or PowerShell:
```
rust-windbg -c "bu rs_f442289d74765418!rs::main;g" target\debug\rs.exe
```
Check privacy of trait items in all contexts
Fixesrust-lang#28514
This is a sufficiently rare scenario and it's currently guarded by `private_in_public` lint, so it shouldn't be a [breaking-change] in practice.
Specify behavior of `write_all` for `ErrorKind::Interrupted` errors
Also spell out that read and write operations should be retried on
`ErrorKind::Interrupted` errors.
Fixesrust-lang#38494.
Add internal accessor methods to io::{Chain, Take}.
Resolvesrust-lang#29067.
Step::replace_one should put a one, not a zero (Issue rust-lang#41492)
Turns out all six of the replace_* impls were backwards.
Fix a copy-paste error in `Instant::sub_duration`
Fixesrust-lang#41514.
@rust-highfive

Copy link
Copy Markdown
Contributor

r? @sfackler

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

@arielb1

Copy link
Copy Markdown
ContributorAuthor

@bors r+ p=10

@bors

bors commented Apr 25, 2017

Copy link
Copy Markdown
Collaborator

📌 Commit a1a19e6 has been approved by arielb1

@bors

bors commented Apr 25, 2017

Copy link
Copy Markdown
Collaborator

⌛ Testing commit a1a19e6 with merge 4e30e41...

@bors

bors commented Apr 25, 2017

Copy link
Copy Markdown
Collaborator

💔 Test failed - status-travis

@aidanhs

aidanhs commented Apr 25, 2017

Copy link
Copy Markdown
Contributor

Seems like a legit failure

[00:37:25] ---- [compile-fail] compile-fail/trait-item-privacy.rs stdout ----
[00:37:25] [00:37:25] error: /checkout/src/test/compile-fail/trait-item-privacy.rs:86: unexpected "error": '86:19: 86:21: the trait `method::C` cannot be made into an object [E0038]'
[00:37:25] [00:37:25] error: /checkout/src/test/compile-fail/trait-item-privacy.rs:88: unexpected "error": '88:7: 88:8: the trait bound `method::C: method::B` is not satisfied [E0277]'
[...]

@arielb1arielb1 closed this Apr 25, 2017
@CentrilCentril added the rollup A PR which is a rollup label Oct 24, 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.

13 participants

@arielb1@rust-highfive@bors@aidanhs@Centril@sfackler@petrochenkov@malbarbo@tbu-@scottmcm@steveklabnik@AndrewGaspar@SergioBenitez