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

fix: terminal freezing on shell_insert_output #4156

Merged
merged 1 commit into from
Oct 26, 2022

Commits on Oct 26, 2022

  1. fix: terminal freezing on shell_insert_output

    This bug occurs on `shell_insert_output` and `shell_append_output`
    commands.
    
    The previous implementation would create a child process using the Rust
    stdlib's `Command` builder. However, when nothing should be piped in
    from the editor, the default value for `stdin` would be used. According
    to the Rust stdlib documentation that is `Stdio::inherit` which will
    make the child process inherit the parent process' stdin. This would
    cause the terminal to freeze.
    
    This change will set the child process' stdin to `Stdio::null` whenever
    it doesn't pipe it. In the `if` statement where this change was made
    there was an extra condition for windows that I am not sure if would
    require some special treatment.
    GabrielDertoni authored and the-mikedavis committed Oct 26, 2022
    Configuration menu
    Copy the full SHA
    ab4296a View commit details
    Browse the repository at this point in the history