diff options
| author | bors <bors@rust-lang.org> | 2016-03-09 18:18:10 -0800 |
|---|---|---|
| committer | bors <bors@rust-lang.org> | 2016-03-09 18:18:10 -0800 |
| commit | 0b9995b24ccc7489e0082199793c300015753dd1 (patch) | |
| tree | 92c841b3b585d37877a5d92facfb74b0bae6eabd /src/libstd/sys/unix/stack_overflow.rs | |
| parent | cc62db802f9cb19606d3bfa4c4df0d0bc438543b (diff) | |
| parent | 7c3038f82477491e20c6f80c0139ddb1f1b912ca (diff) | |
| download | rust-0b9995b24ccc7489e0082199793c300015753dd1.tar.gz rust-0b9995b24ccc7489e0082199793c300015753dd1.zip | |
Auto merge of #31618 - alexcrichton:no-thread-spawns, r=brson
Optimize some functions in std::process * Be sure that `read_to_end` gets directed towards `read_to_end_uninitialized` for all handles * When spawning a child that guaranteed doesn't need a stdin, don't actually create a stdin pipe for that process, instead just redirect it to /dev/null * When calling `wait_with_output`, don't spawn threads to read out the pipes of the child. Instead drain all pipes on the calling thread and *then* wait on the process. Functionally, it is intended that nothing changes as part of this PR --- Note that this was the same as #31613, and even after that it turned out that fixing Windows was easier than I thought! To copy a comment from over there: > As some rationale for this as well, it's always bothered me that we've spawned threads in the standard library for this (seems a bit overkill), and I've also been curious lately as to our why our build times for Windows are so much higher than Unix (on the buildbots we have). I have done basically 0 investigation into why, but I figured it can't help to try to optimize Command::output which I believe is called quite a few times during the test suite.
Diffstat (limited to 'src/libstd/sys/unix/stack_overflow.rs')
0 files changed, 0 insertions, 0 deletions
