fix(embed): handle stdio in server properly

Rename stdin/stdout in the server, so that RPC data won't get corrupted.
This also restores the use of stderr to write directly to the terminal.

1 files changed, 8 insertions, 0 deletions

diff --git a/src/nvim/os/pty_process_unix.c b/src/nvim/os/pty_process_unix.c
index 143f0b3900..cd2150a6a6 100644
--- a/src/nvim/os/pty_process_unix.c
+++ b/src/nvim/os/pty_process_unix.c
@@ -168,6 +168,14 @@ static struct termios termios_default;
 /// @param tty_fd   TTY file descriptor, or -1 if not in a terminal.
 void pty_process_save_termios(int tty_fd)
 {
+  if (embedded_mode) {
+    // TODO(bfredl): currently we cannot use the state of the host terminal in
+    // the server. when the TUI process launches the server, the state has already
+    // changed. we would need to serialize termios_default in the TUI process and
+    // transmit it. Altough, just always using the clean slate of init_termios() might
+    // be preferrable anyway.
+    return;
+  }
   if (tty_fd == -1) {
     return;
   }