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.

Sign up for GitHub

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

Jump to bottom

Add buffering to stdout #115652

Closed
wants to merge 8 commits into from
Closed

Add buffering to stdout #115652

wants to merge 8 commits into from

Commits on Sep 7, 2023

  1. Implement AsFd for sys::stdio structs

    @n-milo
    n-milo committed Sep 7, 2023
    Configuration menu
    Copy the full SHA
    f852f17 View commit details
    Browse the repository at this point in the history

  2. impl AsHandle for sys::stdio structs

    @n-milo
    n-milo committed Sep 7, 2023
    Configuration menu
    Copy the full SHA
    1ea3a8a View commit details
    Browse the repository at this point in the history

  3. Implement IsTerminal for std::stdio structs

    @n-milo
    n-milo committed Sep 7, 2023
    Configuration menu
    Copy the full SHA
    f9ae55c View commit details
    Browse the repository at this point in the history

  4. Create StdoutWriter struct

    @n-milo
    n-milo committed Sep 7, 2023
    Configuration menu
    Copy the full SHA
    abb3867 View commit details
    Browse the repository at this point in the history

  5. Add StdoutWriter::with_capacity

    @n-milo
    n-milo committed Sep 7, 2023
    Configuration menu
    Copy the full SHA
    e5c3a26 View commit details
    Browse the repository at this point in the history

  6. Switch Stdout to use StdoutWriter

    @n-milo
    n-milo committed Sep 7, 2023
    Configuration menu
    Copy the full SHA
    51054d1 View commit details
    Browse the repository at this point in the history

  7. Remove FIXME asking to flush writer on panic 

    I believe we won't need to; the buffer already flushed in
`io::cleanup()` which is called on panic.
    @n-milo
    n-milo committed Sep 7, 2023
    Configuration menu
    Copy the full SHA
    e12c77f View commit details
    Browse the repository at this point in the history

  8. Format fix

    @n-milo
    n-milo committed Sep 7, 2023
    Configuration menu
    Copy the full SHA
    42a1d29 View commit details
    Browse the repository at this point in the history