Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
std: Switch stdout/stderr to buffered by default
Similarly to #12422 which made stdin buffered by default, this commit makes the output streams also buffered by default. Now that buffered writers will flush their contents when they are dropped, I don't believe that there's no reason why the output shouldn't be buffered by default, which is what you want in 90% of cases. As with stdin, there are new stdout_raw() and stderr_raw() functions to get unbuffered streams to stdout/stderr.
- Loading branch information
1 parent
1ee94a1
commit 2cb83fd
Showing
9 changed files
with
41 additions
and
21 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
2cb83fd
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
saw approval from brson
at alexcrichton@2cb83fd
2cb83fd
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
merging alexcrichton/rust/flush-buffered = 2cb83fd into auto
2cb83fd
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
alexcrichton/rust/flush-buffered = 2cb83fd merged ok, testing candidate = 999d55d
2cb83fd
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
all tests pass:
success: http://buildbot.rust-lang.org/builders/auto-mac-32-opt/builds/4553
success: http://buildbot.rust-lang.org/builders/auto-mac-32-nopt-c/builds/2395
success: http://buildbot.rust-lang.org/builders/auto-mac-32-nopt-t/builds/2396
success: http://buildbot.rust-lang.org/builders/auto-mac-64-opt/builds/4551
success: http://buildbot.rust-lang.org/builders/auto-mac-64-nopt-c/builds/3651
success: http://buildbot.rust-lang.org/builders/auto-mac-64-nopt-t/builds/3657
success: http://buildbot.rust-lang.org/builders/auto-linux-32-opt/builds/4566
success: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-c/builds/3654
success: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-t/builds/3660
success: http://buildbot.rust-lang.org/builders/auto-linux-64-opt/builds/4568
success: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-c/builds/3654
success: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-t/builds/3659
success: http://buildbot.rust-lang.org/builders/auto-linux-64-x-android/builds/3728
success: http://buildbot.rust-lang.org/builders/auto-linux-64-x-android-t/builds/1546
success: http://buildbot.rust-lang.org/builders/auto-win-32-opt/builds/4561
success: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-c/builds/3662
success: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-t/builds/3669
success: http://buildbot.rust-lang.org/builders/auto-bsd-64-opt/builds/4327
2cb83fd
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fast-forwarding master to auto = 999d55d