rstream/wstream: Unify structures and simplify API

- Simplify RStream/WStream API and make it more consistent with libuv.
- Move into the event loop layer(event subdirectory)
- Remove uv_helpers module.
- Simplify job/process internal modules/API.
- Unify RStream and WStream into a single structure. This is necessary because
  libuv streams can be readable and writable at the same time(and because the
  uv_helpers.c hack to associate multiple streams with libuv handle was removed)
- Make struct definition public, allowing more flexible/simple memory
  management by users of the module.
- Adapt channel/job modules to cope with the changes.

20 files changed, 121 insertions, 892 deletions

diff --git a/src/nvim/os/input.c b/src/nvim/os/input.c
index 1449c56d59..b0e0f57e60 100644
--- a/src/nvim/os/input.c
+++ b/src/nvim/os/input.c
@@ -7,8 +7,7 @@
 #include "nvim/api/private/defs.h"
 #include "nvim/os/input.h"
 #include "nvim/event/loop.h"
-#include "nvim/os/rstream_defs.h"
-#include "nvim/os/rstream.h"
+#include "nvim/event/rstream.h"
 #include "nvim/ascii.h"
 #include "nvim/vim.h"
 #include "nvim/ui.h"
@@ -30,8 +29,8 @@ typedef enum {
   kInputEof
 } InbufPollResult;
 
-static RStream *read_stream = NULL;
-static RBuffer *read_buffer = NULL, *input_buffer = NULL;
+static Stream read_stream = {.closed = true};
+static RBuffer *input_buffer = NULL;
 static bool input_eof = false;
 static int global_fd = 0;
 
@@ -54,26 +53,23 @@ int input_global_fd(void)
 
 void input_start(int fd)
 {
-  if (read_stream) {
+  if (!read_stream.closed) {
     return;
   }
 
   global_fd = fd;
-  read_buffer = rbuffer_new(READ_BUFFER_SIZE);
-  read_stream = rstream_new(read_cb, read_buffer, NULL);
-  rstream_set_file(read_stream, fd);
-  rstream_start(read_stream);
+  rstream_init_fd(&loop, &read_stream, fd, READ_BUFFER_SIZE, NULL);
+  rstream_start(&read_stream, read_cb);
 }
 
 void input_stop(void)
 {
-  if (!read_stream) {
+  if (read_stream.closed) {
     return;
   }
 
-  rstream_stop(read_stream);
-  rstream_free(read_stream);
-  read_stream = NULL;
+  rstream_stop(&read_stream);
+  stream_close(&read_stream, NULL);
 }
 
 // Low level input function
@@ -309,16 +305,16 @@ static InbufPollResult inbuf_poll(int ms)
   return input_eof ? kInputEof : kInputNone;
 }
 
-static void read_cb(RStream *rstream, RBuffer *buf, void *data, bool at_eof)
+static void read_cb(Stream *stream, RBuffer *buf, void *data, bool at_eof)
 {
   if (at_eof) {
     input_eof = true;
   }
 
-  assert(rbuffer_space(input_buffer) >= rbuffer_size(read_buffer));
-  RBUFFER_UNTIL_EMPTY(read_buffer, ptr, len) {
+  assert(rbuffer_space(input_buffer) >= rbuffer_size(buf));
+  RBUFFER_UNTIL_EMPTY(buf, ptr, len) {
     (void)rbuffer_write(input_buffer, ptr, len);
-    rbuffer_consumed(read_buffer, len);
+    rbuffer_consumed(buf, len);
   }
 }
 
diff --git a/src/nvim/os/job.c b/src/nvim/os/job.c
index 7e90994fb3..71419cefca 100644
--- a/src/nvim/os/job.c
+++ b/src/nvim/os/job.c
@@ -6,15 +6,12 @@
 #include "nvim/event/loop.h"
 #include "nvim/event/time.h"
 #include "nvim/event/signal.h"
-#include "nvim/os/uv_helpers.h"
+#include "nvim/event/rstream.h"
+#include "nvim/event/wstream.h"
 #include "nvim/os/job.h"
 #include "nvim/os/job_defs.h"
 #include "nvim/os/job_private.h"
 #include "nvim/os/pty_process.h"
-#include "nvim/os/rstream.h"
-#include "nvim/os/rstream_defs.h"
-#include "nvim/os/wstream.h"
-#include "nvim/os/wstream_defs.h"
 #include "nvim/os/time.h"
 #include "nvim/vim.h"
 #include "nvim/memory.h"
@@ -29,20 +26,16 @@
 #define KILL_TIMEOUT (TERM_TIMEOUT * 2)
 #define JOB_BUFFER_SIZE 0xFFFF
 
-#define close_job_stream(job, stream, type)                                \
+#define close_job_stream(job, stream)                                      \
   do {                                                                     \
-    if (job->stream) {                                                     \
-      type##stream_free(job->stream);                                      \
-      job->stream = NULL;                                                  \
-      if (!uv_is_closing((uv_handle_t *)job->proc_std##stream)) {          \
-        uv_close((uv_handle_t *)job->proc_std##stream, close_cb);          \
-      }                                                                    \
+    if (!job->stream.closed) {                                             \
+      stream_close(&job->stream, on_##stream_close);                       \
     }                                                                      \
   } while (0)
 
-#define close_job_in(job) close_job_stream(job, in, w)
-#define close_job_out(job) close_job_stream(job, out, r)
-#define close_job_err(job) close_job_stream(job, err, r)
+#define close_job_in(job) close_job_stream(job, in)
+#define close_job_out(job) close_job_stream(job, out)
+#define close_job_err(job) close_job_stream(job, err)
 
 Job *table[MAX_RUNNING_JOBS] = {NULL};
 size_t stop_requests = 0;
@@ -118,63 +111,45 @@ Job *job_start(JobOptions opts, int *status)
   job->refcount = 1;
   job->stopped_time = 0;
   job->term_sent = false;
-  job->in = NULL;
-  job->out = NULL;
-  job->err = NULL;
   job->opts = opts;
   job->closed = false;
-
-  process_init(job);
-
-  if (opts.writable) {
-    handle_set_job((uv_handle_t *)job->proc_stdin, job);
-    job->refcount++;
-  }
-
-  if (opts.stdout_cb) {
-    handle_set_job((uv_handle_t *)job->proc_stdout, job);
-    job->refcount++;
-  }
-
-  if (opts.stderr_cb) {
-    handle_set_job((uv_handle_t *)job->proc_stderr, job);
-    job->refcount++;
-  }
+  job->in.closed = true;
+  job->out.closed = true;
+  job->err.closed = true;
 
   // Spawn the job
   if (!process_spawn(job)) {
-    if (opts.writable) {
-      uv_close((uv_handle_t *)job->proc_stdin, close_cb);
+    if (job->opts.writable) {
+      uv_close((uv_handle_t *)job->proc_stdin, NULL);
     }
-    if (opts.stdout_cb) {
-      uv_close((uv_handle_t *)job->proc_stdout, close_cb);
+    if (job->opts.stdout_cb) {
+      uv_close((uv_handle_t *)job->proc_stdout, NULL);
     }
-    if (opts.stderr_cb) {
-      uv_close((uv_handle_t *)job->proc_stderr, close_cb);
+    if (job->opts.stderr_cb) {
+      uv_close((uv_handle_t *)job->proc_stderr, NULL);
     }
     process_close(job);
     loop_poll_events(&loop, 0);
-    // Manually invoke the close_cb to free the job resources
     *status = -1;
     return NULL;
   }
 
   if (opts.writable) {
-    job->in = wstream_new(opts.maxmem);
-    wstream_set_stream(job->in, job->proc_stdin);
+    job->refcount++;
+    wstream_init_stream(&job->in, job->proc_stdin, opts.maxmem, job);
   }
 
   // Start the readable streams
   if (opts.stdout_cb) {
-    job->out = rstream_new(read_cb, rbuffer_new(JOB_BUFFER_SIZE), job);
-    rstream_set_stream(job->out, job->proc_stdout);
-    rstream_start(job->out);
+    job->refcount++;
+    rstream_init_stream(&job->out, job->proc_stdout, JOB_BUFFER_SIZE, job);
+    rstream_start(&job->out, read_cb);
   }
 
   if (opts.stderr_cb) {
-    job->err = rstream_new(read_cb, rbuffer_new(JOB_BUFFER_SIZE), job);
-    rstream_set_stream(job->err, job->proc_stderr);
-    rstream_start(job->err);
+    job->refcount++;
+    rstream_init_stream(&job->err, job->proc_stderr, JOB_BUFFER_SIZE, job);
+    rstream_start(&job->err, read_cb);
   }
   // Save the job to the table
   table[i] = job;
@@ -217,7 +192,7 @@ void job_stop(Job *job)
     // Close the job's stdin. If the job doesn't close its own stdout/stderr,
     // they will be closed when the job exits(possibly due to being terminated
     // after a timeout)
-    close_job_in(job);
+    job_close_in(job);
   }
 
   if (!stop_requests++) {
@@ -315,9 +290,9 @@ void job_close_err(Job *job) FUNC_ATTR_NONNULL_ALL
 /// @param job The job instance
 /// @param cb The function that will be called on write completion or
 ///        failure. It will be called with the job as the `data` argument.
-void job_write_cb(Job *job, wstream_cb cb) FUNC_ATTR_NONNULL_ALL
+void job_write_cb(Job *job, stream_write_cb cb) FUNC_ATTR_NONNULL_ALL
 {
-  wstream_set_write_cb(job->in, cb, job);
+  wstream_set_write_cb(&job->in, cb);
 }
 
 /// Writes data to the job's stdin. This is a non-blocking operation, it
@@ -329,7 +304,7 @@ void job_write_cb(Job *job, wstream_cb cb) FUNC_ATTR_NONNULL_ALL
 ///         to the job stream failed (possibly because the OS buffer is full)
 bool job_write(Job *job, WBuffer *buffer)
 {
-  return wstream_write(job->in, buffer);
+  return wstream_write(&job->in, buffer);
 }
 
 /// Get the job id
@@ -405,26 +380,26 @@ static void job_stop_timer_cb(TimeWatcher *watcher, void *data)
 }
 
 // Wraps the call to std{out,err}_cb and emits a JobExit event if necessary.
-static void read_cb(RStream *rstream, RBuffer *buf, void *data, bool eof)
+static void read_cb(Stream *stream, RBuffer *buf, void *data, bool eof)
 {
   Job *job = data;
 
-  if (rstream == job->out) {
-    job->opts.stdout_cb(rstream, buf, data, eof);
+  if (stream == &job->out) {
+    job->opts.stdout_cb(stream, buf, data, eof);
     if (eof) {
       close_job_out(job);
     }
   } else {
-    job->opts.stderr_cb(rstream, buf, data, eof);
+    job->opts.stderr_cb(stream, buf, data, eof);
     if (eof) {
       close_job_err(job);
     }
   }
 }
 
-static void close_cb(uv_handle_t *handle)
+static void on_stream_close(Stream *stream, void *data)
 {
-  job_decref(handle_get_job(handle));
+  job_decref(data);
 }
 
 static void job_exited(Event event)
diff --git a/src/nvim/os/job.h b/src/nvim/os/job.h
index ed102666d0..2f8bf79a31 100644
--- a/src/nvim/os/job.h
+++ b/src/nvim/os/job.h
@@ -10,10 +10,9 @@
 #include <stdint.h>
 #include <stdbool.h>
 
-#include "nvim/os/rstream_defs.h"
 #include "nvim/os/job_defs.h"
-#include "nvim/os/wstream.h"
-#include "nvim/os/wstream_defs.h"
+#include "nvim/event/rstream.h"
+#include "nvim/event/wstream.h"
 
 #ifdef INCLUDE_GENERATED_DECLARATIONS
 # include "os/job.h.generated.h"
diff --git a/src/nvim/os/job_defs.h b/src/nvim/os/job_defs.h
index 7fee900ac0..ea7a326404 100644
--- a/src/nvim/os/job_defs.h
+++ b/src/nvim/os/job_defs.h
@@ -2,8 +2,9 @@
 #define NVIM_OS_JOB_DEFS_H
 
 #include <uv.h>
-#include "nvim/os/rstream_defs.h"
-#include "nvim/os/wstream_defs.h"
+
+#include "nvim/event/rstream.h"
+#include "nvim/event/wstream.h"
 
 #define MAX_RUNNING_JOBS 100
 typedef struct job Job;
@@ -29,10 +30,10 @@ typedef struct {
   bool writable;
   // Callback that will be invoked when data is available on stdout. If NULL
   // stdout will be redirected to /dev/null.
-  rstream_cb stdout_cb;
+  stream_read_cb stdout_cb;
   // Callback that will be invoked when data is available on stderr. If NULL
   // stderr will be redirected to /dev/null.
-  rstream_cb  stderr_cb;
+  stream_read_cb stderr_cb;
   // Callback that will be invoked when the job has exited and will not send
   // data
   job_exit_cb exit_cb;
diff --git a/src/nvim/os/job_private.h b/src/nvim/os/job_private.h
index b90f2d0171..6bdb24e6cd 100644
--- a/src/nvim/os/job_private.h
+++ b/src/nvim/os/job_private.h
@@ -6,8 +6,8 @@
 #include <uv.h>
 
 #include "nvim/event/time.h"
-#include "nvim/os/rstream_defs.h"
-#include "nvim/os/wstream_defs.h"
+#include "nvim/event/rstream.h"
+#include "nvim/event/wstream.h"
 #include "nvim/os/pipe_process.h"
 #include "nvim/os/pty_process.h"
 #include "nvim/os/shell.h"
@@ -28,14 +28,15 @@ struct job {
   uint64_t stopped_time;
   // If SIGTERM was already sent to the job(only send one before SIGKILL)
   bool term_sent;
-  // Readable streams(std{out,err})
-  RStream *out, *err;
-  // Writable stream(stdin)
-  WStream *in;
+  // stdio streams(std{in,out,err})
+  Stream in, out, err;
   // Libuv streams representing stdin/stdout/stderr
   uv_stream_t *proc_stdin, *proc_stdout, *proc_stderr;
   // Extra data set by the process spawner
-  void *process;
+  union {
+    UvProcess uv;
+    PtyProcess pty;
+  } process;
   // If process_close has been called on this job
   bool closed;
   // Startup options
@@ -51,15 +52,6 @@ static inline bool process_spawn(Job *job)
   return job->opts.pty ? pty_process_spawn(job) : pipe_process_spawn(job);
 }
 
-static inline void process_init(Job *job)
-{
-  if (job->opts.pty) {
-    pty_process_init(job);
-  } else {
-    pipe_process_init(job);
-  }
-}
-
 static inline void process_close(Job *job)
 {
   if (job->closed) {
@@ -73,15 +65,6 @@ static inline void process_close(Job *job)
   }
 }
 
-static inline void process_destroy(Job *job)
-{
-  if (job->opts.pty) {
-    pty_process_destroy(job);
-  } else {
-    pipe_process_destroy(job);
-  }
-}
-
 static inline void job_exit_callback(Job *job)
 {
   // Free the slot now, 'exit_cb' may want to start another job to replace
@@ -106,11 +89,10 @@ static inline void job_decref(Job *job)
     // Invoke the exit_cb
     job_exit_callback(job);
     // Free all memory allocated for the job
-    xfree(job->proc_stdin->data);
-    xfree(job->proc_stdout->data);
-    xfree(job->proc_stderr->data);
     shell_free_argv(job->opts.argv);
-    process_destroy(job);
+    if (job->opts.pty) {
+      xfree(job->opts.term_name);
+    }
     xfree(job);
   }
 }
diff --git a/src/nvim/os/os.h b/src/nvim/os/os.h
index 3dd099890c..69bd1ff4fd 100644
--- a/src/nvim/os/os.h
+++ b/src/nvim/os/os.h
@@ -12,7 +12,6 @@
 # include "os/mem.h.generated.h"
 # include "os/env.h.generated.h"
 # include "os/users.h.generated.h"
-# include "os/stream.h.generated.h"
 #endif
 
 #endif  // NVIM_OS_OS_H
diff --git a/src/nvim/os/pipe_process.c b/src/nvim/os/pipe_process.c
index 9980cf7c56..1160015c34 100644
--- a/src/nvim/os/pipe_process.c
+++ b/src/nvim/os/pipe_process.c
@@ -3,7 +3,6 @@
 
 #include <uv.h>
 
-#include "nvim/os/uv_helpers.h"
 #include "nvim/os/job.h"
 #include "nvim/os/job_defs.h"
 #include "nvim/os/job_private.h"
@@ -16,17 +15,9 @@
 # include "os/pipe_process.c.generated.h"
 #endif
 
-typedef struct {
-  // Structures for process spawning/management used by libuv
-  uv_process_t proc;
-  uv_process_options_t proc_opts;
-  uv_stdio_container_t stdio[3];
-  uv_pipe_t proc_stdin, proc_stdout, proc_stderr;
-} UvProcess;
-
-void pipe_process_init(Job *job)
+bool pipe_process_spawn(Job *job)
 {
-  UvProcess *pipeproc = xmalloc(sizeof(UvProcess));
+  UvProcess *pipeproc = &job->process.uv;
   pipeproc->proc_opts.file = job->opts.argv[0];
   pipeproc->proc_opts.args = job->opts.argv;
   pipeproc->proc_opts.stdio = pipeproc->stdio;
@@ -45,7 +36,7 @@ void pipe_process_init(Job *job)
   pipeproc->stdio[1].flags = UV_IGNORE;
   pipeproc->stdio[2].flags = UV_IGNORE;
 
-  handle_set_job((uv_handle_t *)&pipeproc->proc, job);
+  pipeproc->proc.data = job;
 
   if (job->opts.writable) {
     uv_pipe_init(&loop.uv, &pipeproc->proc_stdin, 0);
@@ -68,20 +59,6 @@ void pipe_process_init(Job *job)
   job->proc_stdin = (uv_stream_t *)&pipeproc->proc_stdin;
   job->proc_stdout = (uv_stream_t *)&pipeproc->proc_stdout;
   job->proc_stderr = (uv_stream_t *)&pipeproc->proc_stderr;
-  job->process = pipeproc;
-}
-
-void pipe_process_destroy(Job *job)
-{
-  UvProcess *pipeproc = job->process;
-  xfree(pipeproc->proc.data);
-  xfree(pipeproc);
-  job->process = NULL;
-}
-
-bool pipe_process_spawn(Job *job)
-{
-  UvProcess *pipeproc = job->process;
 
   if (uv_spawn(&loop.uv, &pipeproc->proc, &pipeproc->proc_opts) != 0) {
     return false;
@@ -93,20 +70,19 @@ bool pipe_process_spawn(Job *job)
 
 void pipe_process_close(Job *job)
 {
-  UvProcess *pipeproc = job->process;
-  uv_close((uv_handle_t *)&pipeproc->proc, close_cb);
+  uv_close((uv_handle_t *)&job->process.uv.proc, close_cb);
 }
 
 static void exit_cb(uv_process_t *proc, int64_t status, int term_signal)
 {
-  Job *job = handle_get_job((uv_handle_t *)proc);
+  Job *job = proc->data;
   job->status = (int)status;
   pipe_process_close(job);
 }
 
 static void close_cb(uv_handle_t *handle)
 {
-  Job *job = handle_get_job(handle);
+  Job *job = handle->data;
   job_close_streams(job);
   job_decref(job);
 }
diff --git a/src/nvim/os/pipe_process.h b/src/nvim/os/pipe_process.h
index 17a4255ddc..65e5cfa78f 100644
--- a/src/nvim/os/pipe_process.h
+++ b/src/nvim/os/pipe_process.h
@@ -1,6 +1,16 @@
 #ifndef NVIM_OS_PIPE_PROCESS_H
 #define NVIM_OS_PIPE_PROCESS_H
 
+#include <uv.h>
+
+typedef struct {
+  // Structures for process spawning/management used by libuv
+  uv_process_t proc;
+  uv_process_options_t proc_opts;
+  uv_stdio_container_t stdio[3];
+  uv_pipe_t proc_stdin, proc_stdout, proc_stderr;
+} UvProcess;
+
 #ifdef INCLUDE_GENERATED_DECLARATIONS
 # include "os/pipe_process.h.generated.h"
 #endif
diff --git a/src/nvim/os/pty_process.c b/src/nvim/os/pty_process.c
index ca74ebfddd..a8fff2a61d 100644
--- a/src/nvim/os/pty_process.c
+++ b/src/nvim/os/pty_process.c
@@ -33,15 +33,12 @@
 # include "os/pty_process.c.generated.h"
 #endif
 
-typedef struct {
-  struct winsize winsize;
-  uv_pipe_t proc_stdin, proc_stdout, proc_stderr;
-  int tty_fd;
-} PtyProcess;
+static const unsigned int KILL_RETRIES = 5;
+static const unsigned int KILL_TIMEOUT = 2;  // seconds
 
-void pty_process_init(Job *job) FUNC_ATTR_NONNULL_ALL
+bool pty_process_spawn(Job *job) FUNC_ATTR_NONNULL_ALL
 {
-  PtyProcess *ptyproc = xmalloc(sizeof(PtyProcess));
+  PtyProcess *ptyproc = &job->process.pty;
   ptyproc->tty_fd = -1;
 
   if (job->opts.writable) {
@@ -62,41 +59,8 @@ void pty_process_init(Job *job) FUNC_ATTR_NONNULL_ALL
   job->proc_stdin = (uv_stream_t *)&ptyproc->proc_stdin;
   job->proc_stdout = (uv_stream_t *)&ptyproc->proc_stdout;
   job->proc_stderr = (uv_stream_t *)&ptyproc->proc_stderr;
-  job->process = ptyproc;
-}
-
-void pty_process_destroy(Job *job) FUNC_ATTR_NONNULL_ALL
-{
-  xfree(job->opts.term_name);
-  xfree(job->process);
-  job->process = NULL;
-}
-
-static bool set_pipe_duplicating_descriptor(int fd, uv_pipe_t *pipe)
-  FUNC_ATTR_NONNULL_ALL
-{
-  int fd_dup = dup(fd);
-  if (fd_dup < 0) {
-    ELOG("Failed to dup descriptor %d: %s", fd, strerror(errno));
-    return false;
-  }
-  int uv_result = uv_pipe_open(pipe, fd_dup);
-  if (uv_result) {
-    ELOG("Failed to set pipe to descriptor %d: %s",
-         fd_dup, uv_strerror(uv_result));
-    close(fd_dup);
-    return false;
-  }
-  return true;
-}
-
-static const unsigned int KILL_RETRIES = 5;
-static const unsigned int KILL_TIMEOUT = 2;  // seconds
 
-bool pty_process_spawn(Job *job) FUNC_ATTR_NONNULL_ALL
-{
   int master;
-  PtyProcess *ptyproc = job->process;
   ptyproc->winsize = (struct winsize){job->opts.height, job->opts.width, 0, 0};
   struct termios termios;
   init_termios(&termios);
@@ -158,6 +122,24 @@ error:
   return false;
 }
 
+static bool set_pipe_duplicating_descriptor(int fd, uv_pipe_t *pipe)
+  FUNC_ATTR_NONNULL_ALL
+{
+  int fd_dup = dup(fd);
+  if (fd_dup < 0) {
+    ELOG("Failed to dup descriptor %d: %s", fd, strerror(errno));
+    return false;
+  }
+  int uv_result = uv_pipe_open(pipe, fd_dup);
+  if (uv_result) {
+    ELOG("Failed to set pipe to descriptor %d: %s",
+         fd_dup, uv_strerror(uv_result));
+    close(fd_dup);
+    return false;
+  }
+  return true;
+}
+
 void pty_process_close(Job *job) FUNC_ATTR_NONNULL_ALL
 {
   pty_process_close_master(job);
@@ -167,7 +149,7 @@ void pty_process_close(Job *job) FUNC_ATTR_NONNULL_ALL
 
 void pty_process_close_master(Job *job) FUNC_ATTR_NONNULL_ALL
 {
-  PtyProcess *ptyproc = job->process;
+  PtyProcess *ptyproc = &job->process.pty;
   if (ptyproc->tty_fd >= 0) {
     close(ptyproc->tty_fd);
     ptyproc->tty_fd = -1;
@@ -177,7 +159,7 @@ void pty_process_close_master(Job *job) FUNC_ATTR_NONNULL_ALL
 void pty_process_resize(Job *job, uint16_t width, uint16_t height)
   FUNC_ATTR_NONNULL_ALL
 {
-  PtyProcess *ptyproc = job->process;
+  PtyProcess *ptyproc = &job->process.pty;
   ptyproc->winsize = (struct winsize){height, width, 0, 0};
   ioctl(ptyproc->tty_fd, TIOCSWINSZ, &ptyproc->winsize);
 }
diff --git a/src/nvim/os/pty_process.h b/src/nvim/os/pty_process.h
index 62fcd1671f..b5a2eba8b3 100644
--- a/src/nvim/os/pty_process.h
+++ b/src/nvim/os/pty_process.h
@@ -1,6 +1,16 @@
 #ifndef NVIM_OS_PTY_PROCESS_H
 #define NVIM_OS_PTY_PROCESS_H
 
+#include <sys/ioctl.h>
+
+#include <uv.h>
+
+typedef struct {
+  struct winsize winsize;
+  uv_pipe_t proc_stdin, proc_stdout, proc_stderr;
+  int tty_fd;
+} PtyProcess;
+
 #ifdef INCLUDE_GENERATED_DECLARATIONS
 # include "os/pty_process.h.generated.h"
 #endif
diff --git a/src/nvim/os/rstream.c b/src/nvim/os/rstream.c
deleted file mode 100644
index dd91c2777e..0000000000
--- a/src/nvim/os/rstream.c
+++ /dev/null
@@ -1,253 +0,0 @@
-#include <assert.h>
-#include <stdint.h>
-#include <stdbool.h>
-#include <stdlib.h>
-
-#include <uv.h>
-
-#include "nvim/os/uv_helpers.h"
-#include "nvim/os/rstream_defs.h"
-#include "nvim/os/rstream.h"
-#include "nvim/ascii.h"
-#include "nvim/vim.h"
-#include "nvim/memory.h"
-#include "nvim/log.h"
-#include "nvim/misc1.h"
-
-struct rstream {
-  void *data;
-  uv_buf_t uvbuf;
-  size_t fpos;
-  RBuffer *buffer;
-  uv_stream_t *stream;
-  uv_idle_t *fread_idle;
-  uv_handle_type file_type;
-  uv_file fd;
-  rstream_cb cb;
-  bool free_handle;
-};
-
-#ifdef INCLUDE_GENERATED_DECLARATIONS
-# include "os/rstream.c.generated.h"
-#endif
-
-/// Creates a new RStream instance. A RStream encapsulates all the boilerplate
-/// necessary for reading from a libuv stream.
-///
-/// @param cb A function that will be called whenever some data is available
-///        for reading with `rstream_read`
-/// @param buffer RBuffer instance to associate with the RStream
-/// @param data Some state to associate with the `RStream` instance
-/// @return The newly-allocated `RStream` instance
-RStream * rstream_new(rstream_cb cb, RBuffer *buffer, void *data)
-{
-  RStream *rv = xmalloc(sizeof(RStream));
-  buffer->data = rv;
-  buffer->full_cb = on_rbuffer_full;
-  buffer->nonfull_cb = on_rbuffer_nonfull;
-  rv->buffer = buffer;
-  rv->fpos = 0;
-  rv->data = data;
-  rv->cb = cb;
-  rv->stream = NULL;
-  rv->fread_idle = NULL;
-  rv->free_handle = false;
-  rv->file_type = UV_UNKNOWN_HANDLE;
-
-  return rv;
-}
-
-static void on_rbuffer_full(RBuffer *buf, void *data)
-{
-  rstream_stop(data);
-}
-
-static void on_rbuffer_nonfull(RBuffer *buf, void *data)
-{
-  rstream_start(data);
-}
-
-/// Frees all memory allocated for a RStream instance
-///
-/// @param rstream The `RStream` instance
-void rstream_free(RStream *rstream)
-{
-  if (rstream->free_handle) {
-    if (rstream->fread_idle != NULL) {
-      uv_close((uv_handle_t *)rstream->fread_idle, close_cb);
-    } else {
-      uv_close((uv_handle_t *)rstream->stream, close_cb);
-    }
-  }
-
-  rbuffer_free(rstream->buffer);
-  xfree(rstream);
-}
-
-/// Sets the underlying `uv_stream_t` instance
-///
-/// @param rstream The `RStream` instance
-/// @param stream The new `uv_stream_t` instance
-void rstream_set_stream(RStream *rstream, uv_stream_t *stream)
-{
-  handle_set_rstream((uv_handle_t *)stream, rstream);
-  rstream->stream = stream;
-}
-
-/// Sets the underlying file descriptor that will be read from. Only pipes
-/// and regular files are supported for now.
-///
-/// @param rstream The `RStream` instance
-/// @param file The file descriptor
-void rstream_set_file(RStream *rstream, uv_file file)
-{
-  rstream->file_type = uv_guess_handle(file);
-
-  if (rstream->free_handle) {
-    // If this is the second time we're calling this function, free the
-    // previously allocated memory
-    if (rstream->fread_idle != NULL) {
-      uv_close((uv_handle_t *)rstream->fread_idle, close_cb);
-      rstream->fread_idle = NULL;
-    } else {
-      uv_close((uv_handle_t *)rstream->stream, close_cb);
-      rstream->stream = NULL;
-    }
-  }
-
-  if (rstream->file_type == UV_FILE) {
-    // Non-blocking file reads are simulated with an idle handle that reads
-    // in chunks of rstream->buffer_size, giving time for other events to
-    // be processed between reads.
-    rstream->fread_idle = xmalloc(sizeof(uv_idle_t));
-    uv_idle_init(&loop.uv, rstream->fread_idle);
-    rstream->fread_idle->data = NULL;
-    handle_set_rstream((uv_handle_t *)rstream->fread_idle, rstream);
-  } else {
-    // Only pipes are supported for now
-    assert(rstream->file_type == UV_NAMED_PIPE
-        || rstream->file_type == UV_TTY);
-    rstream->stream = xmalloc(sizeof(uv_pipe_t));
-    uv_pipe_init(&loop.uv, (uv_pipe_t *)rstream->stream, 0);
-    uv_pipe_open((uv_pipe_t *)rstream->stream, file);
-    rstream->stream->data = NULL;
-    handle_set_rstream((uv_handle_t *)rstream->stream, rstream);
-  }
-
-  rstream->fd = file;
-  rstream->free_handle = true;
-}
-
-/// Starts watching for events from a `RStream` instance.
-///
-/// @param rstream The `RStream` instance
-void rstream_start(RStream *rstream)
-{
-  if (rstream->file_type == UV_FILE) {
-    uv_idle_start(rstream->fread_idle, fread_idle_cb);
-  } else {
-    uv_read_start(rstream->stream, alloc_cb, read_cb);
-  }
-}
-
-/// Stops watching for events from a `RStream` instance.
-///
-/// @param rstream The `RStream` instance
-void rstream_stop(RStream *rstream)
-{
-  if (rstream->file_type == UV_FILE) {
-    uv_idle_stop(rstream->fread_idle);
-  } else {
-    uv_read_stop(rstream->stream);
-  }
-}
-
-// Callbacks used by libuv
-
-// Called by libuv to allocate memory for reading.
-static void alloc_cb(uv_handle_t *handle, size_t suggested, uv_buf_t *buf)
-{
-  RStream *rstream = handle_get_rstream(handle);
-  buf->base = rbuffer_write_ptr(rstream->buffer, &buf->len);
-}
-
-// Callback invoked by libuv after it copies the data into the buffer provided
-// by `alloc_cb`. This is also called on EOF or when `alloc_cb` returns a
-// 0-length buffer.
-static void read_cb(uv_stream_t *stream, ssize_t cnt, const uv_buf_t *buf)
-{
-  RStream *rstream = handle_get_rstream((uv_handle_t *)stream);
-
-  if (cnt <= 0) {
-    if (cnt != UV_ENOBUFS
-        // cnt == 0 means libuv asked for a buffer and decided it wasn't needed:
-        // http://docs.libuv.org/en/latest/stream.html#c.uv_read_start.
-        //
-        // We don't need to do anything with the RBuffer because the next call
-        // to `alloc_cb` will return the same unused pointer(`rbuffer_produced`
-        // won't be called)
-        && cnt != 0) {
-      DLOG("Closing RStream(%p) because of %s(%zd)", rstream,
-           uv_strerror((int)cnt), cnt);
-      // Read error or EOF, either way stop the stream and invoke the callback
-      // with eof == true
-      uv_read_stop(stream);
-      rstream->cb(rstream, rstream->buffer, rstream->data, true);
-    }
-    return;
-  }
-
-  // at this point we're sure that cnt is positive, no error occurred
-  size_t nread = (size_t)cnt;
-  // Data was already written, so all we need is to update 'wpos' to reflect
-  // the space actually used in the buffer.
-  rbuffer_produced(rstream->buffer, nread);
-  rstream->cb(rstream, rstream->buffer, rstream->data, false);
-}
-
-// Called by the by the 'idle' handle to emulate a reading event
-static void fread_idle_cb(uv_idle_t *handle)
-{
-  uv_fs_t req;
-  RStream *rstream = handle_get_rstream((uv_handle_t *)handle);
-
-  rstream->uvbuf.base = rbuffer_write_ptr(rstream->buffer, &rstream->uvbuf.len);
-
-  // the offset argument to uv_fs_read is int64_t, could someone really try
-  // to read more than 9 quintillion (9e18) bytes?
-  // upcast is meant to avoid tautological condition warning on 32 bits
-  uintmax_t fpos_intmax = rstream->fpos;
-  if (fpos_intmax > INT64_MAX) {
-    ELOG("stream offset overflow");
-    preserve_exit();
-  }
-
-  // Synchronous read
-  uv_fs_read(
-      &loop.uv,
-      &req,
-      rstream->fd,
-      &rstream->uvbuf,
-      1,
-      (int64_t) rstream->fpos,
-      NULL);
-
-  uv_fs_req_cleanup(&req);
-
-  if (req.result <= 0) {
-    uv_idle_stop(rstream->fread_idle);
-    rstream->cb(rstream, rstream->buffer, rstream->data, true);
-    return;
-  }
-
-  // no errors (req.result (ssize_t) is positive), it's safe to cast.
-  size_t nread = (size_t) req.result;
-  rbuffer_produced(rstream->buffer, nread);
-  rstream->fpos += nread;
-}
-
-static void close_cb(uv_handle_t *handle)
-{
-  xfree(handle->data);
-  xfree(handle);
-}
diff --git a/src/nvim/os/rstream.h b/src/nvim/os/rstream.h
deleted file mode 100644
index 7da77b33fa..0000000000
--- a/src/nvim/os/rstream.h
+++ /dev/null
@@ -1,12 +0,0 @@
-#ifndef NVIM_OS_RSTREAM_H
-#define NVIM_OS_RSTREAM_H
-
-#include <stdbool.h>
-#include <stdint.h>
-#include <uv.h>
-#include "nvim/os/rstream_defs.h"
-
-#ifdef INCLUDE_GENERATED_DECLARATIONS
-# include "os/rstream.h.generated.h"
-#endif
-#endif  // NVIM_OS_RSTREAM_H
diff --git a/src/nvim/os/rstream_defs.h b/src/nvim/os/rstream_defs.h
deleted file mode 100644
index 45dced0b62..0000000000
--- a/src/nvim/os/rstream_defs.h
+++ /dev/null
@@ -1,20 +0,0 @@
-#ifndef NVIM_OS_RSTREAM_DEFS_H
-#define NVIM_OS_RSTREAM_DEFS_H
-
-#include <stdbool.h>
-
-#include "nvim/rbuffer.h"
-
-typedef struct rstream RStream;
-
-/// Type of function called when the RStream receives data
-///
-/// @param rstream The RStream instance
-/// @param rbuffer The associated RBuffer instance
-/// @param data State associated with the RStream instance
-/// @param eof If the stream reached EOF.
-typedef void (*rstream_cb)(RStream *rstream, RBuffer *buf, void *data,
-    bool eof);
-
-#endif  // NVIM_OS_RSTREAM_DEFS_H
-
diff --git a/src/nvim/os/shell.c b/src/nvim/os/shell.c
index 9c0d5fca67..04ac9f1c03 100644
--- a/src/nvim/os/shell.c
+++ b/src/nvim/os/shell.c
@@ -10,7 +10,7 @@
 #include "nvim/log.h"
 #include "nvim/event/loop.h"
 #include "nvim/os/job.h"
-#include "nvim/os/rstream.h"
+#include "nvim/event/rstream.h"
 #include "nvim/os/shell.h"
 #include "nvim/os/signal.h"
 #include "nvim/types.h"
@@ -189,7 +189,7 @@ static int do_os_system(char **argv,
 {
   // the output buffer
   DynamicBuffer buf = DYNAMIC_BUFFER_INIT;
-  rstream_cb data_cb = system_data_cb;
+  stream_read_cb data_cb = system_data_cb;
   if (nread) {
     *nread = 0;
   }
@@ -283,7 +283,7 @@ static void dynamic_buffer_ensure(DynamicBuffer *buf, size_t desired)
   buf->data = xrealloc(buf->data, buf->cap);
 }
 
-static void system_data_cb(RStream *rstream, RBuffer *buf, void *data, bool eof)
+static void system_data_cb(Stream *stream, RBuffer *buf, void *data, bool eof)
 {
   Job *job = data;
   DynamicBuffer *dbuf = job_data(job);
@@ -294,7 +294,7 @@ static void system_data_cb(RStream *rstream, RBuffer *buf, void *data, bool eof)
   dbuf->len += nread;
 }
 
-static void out_data_cb(RStream *rstream, RBuffer *buf, void *data, bool eof)
+static void out_data_cb(Stream *stream, RBuffer *buf, void *data, bool eof)
 {
   RBUFFER_UNTIL_EMPTY(buf, ptr, len) {
     size_t written = write_output(ptr, len, false,
@@ -470,7 +470,7 @@ static size_t write_output(char *output, size_t remaining, bool to_buffer,
   return (size_t)(output - start);
 }
 
-static void shell_write_cb(WStream *wstream, void *data, int status)
+static void shell_write_cb(Stream *stream, void *data, int status)
 {
   Job *job = data;
   job_close_in(job);
diff --git a/src/nvim/os/stream.c b/src/nvim/os/stream.c
deleted file mode 100644
index 0c448872c3..0000000000
--- a/src/nvim/os/stream.c
+++ /dev/null
@@ -1,30 +0,0 @@
-// Functions for working with stdio streams (as opposed to RStream/WStream).
-
-#include <stdio.h>
-#include <stdbool.h>
-
-#include <uv.h>
-
-#ifdef INCLUDE_GENERATED_DECLARATIONS
-# include "os/stream.c.generated.h"
-#endif
-
-/// Sets the stream associated with `fd` to "blocking" mode.
-///
-/// @return `0` on success, or `-errno` on failure.
-int stream_set_blocking(int fd, bool blocking)
-{
-  // Private loop to avoid conflict with existing watcher(s):
-  //    uv__io_stop: Assertion `loop->watchers[w->fd] == w' failed.
-  uv_loop_t loop;
-  uv_pipe_t stream;
-  uv_loop_init(&loop);
-  uv_pipe_init(&loop, &stream, 0);
-  uv_pipe_open(&stream, fd);
-  int retval = uv_stream_set_blocking((uv_stream_t *)&stream, blocking);
-  uv_close((uv_handle_t *)&stream, NULL);
-  uv_run(&loop, UV_RUN_NOWAIT);  // not necessary, but couldn't hurt.
-  uv_loop_close(&loop);
-  return retval;
-}
-
diff --git a/src/nvim/os/uv_helpers.c b/src/nvim/os/uv_helpers.c
deleted file mode 100644
index 89687bdac7..0000000000
--- a/src/nvim/os/uv_helpers.c
+++ /dev/null
@@ -1,98 +0,0 @@
-#include <assert.h>
-#include <uv.h>
-
-#include "nvim/os/uv_helpers.h"
-#include "nvim/vim.h"
-#include "nvim/memory.h"
-
-/// Common structure that will always be assigned to the `data` field of
-/// libuv handles. It has fields for many types of pointers, and allow a single
-/// handle to contain data from many sources
-typedef struct {
-  WStream *wstream;
-  RStream *rstream;
-  Job *job;
-} HandleData;
-
-
-#ifdef INCLUDE_GENERATED_DECLARATIONS
-# include "os/uv_helpers.c.generated.h"
-#endif
-
-/// Gets the RStream instance associated with a libuv handle
-///
-/// @param handle libuv handle
-/// @return the RStream pointer
-RStream *handle_get_rstream(uv_handle_t *handle)
-{
-  RStream *rv = init(handle)->rstream;
-  assert(rv != NULL);
-  return rv;
-}
-
-/// Associates a RStream instance with a libuv handle
-///
-/// @param handle libuv handle
-/// @param rstream the RStream pointer
-void handle_set_rstream(uv_handle_t *handle, RStream *rstream)
-{
-  init(handle)->rstream = rstream;
-}
-
-/// Gets the WStream instance associated with a libuv handle
-///
-/// @param handle libuv handle
-/// @return the WStream pointer
-WStream *handle_get_wstream(uv_handle_t *handle)
-{
-  WStream *rv = init(handle)->wstream;
-  assert(rv != NULL);
-  return rv;
-}
-
-/// Associates a WStream instance with a libuv handle
-///
-/// @param handle libuv handle
-/// @param wstream the WStream pointer
-void handle_set_wstream(uv_handle_t *handle, WStream *wstream)
-{
-  HandleData *data = init(handle);
-  data->wstream = wstream;
-}
-
-/// Gets the Job instance associated with a libuv handle
-///
-/// @param handle libuv handle
-/// @return the Job pointer
-Job *handle_get_job(uv_handle_t *handle)
-{
-  Job *rv = init(handle)->job;
-  assert(rv != NULL);
-  return rv;
-}
-
-/// Associates a Job instance with a libuv handle
-///
-/// @param handle libuv handle
-/// @param job the Job pointer
-void handle_set_job(uv_handle_t *handle, Job *job)
-{
-  init(handle)->job = job;
-}
-
-static HandleData *init(uv_handle_t *handle)
-{
-  HandleData *rv;
-
-  if (handle->data == NULL) {
-    rv = xmalloc(sizeof(HandleData));
-    rv->rstream = NULL;
-    rv->wstream = NULL;
-    rv->job = NULL;
-    handle->data = rv;
-  } else {
-    rv = handle->data;
-  }
-
-  return rv;
-}
diff --git a/src/nvim/os/uv_helpers.h b/src/nvim/os/uv_helpers.h
deleted file mode 100644
index b49656bcb8..0000000000
--- a/src/nvim/os/uv_helpers.h
+++ /dev/null
@@ -1,13 +0,0 @@
-#ifndef NVIM_OS_UV_HELPERS_H
-#define NVIM_OS_UV_HELPERS_H
-
-#include <uv.h>
-
-#include "nvim/os/wstream_defs.h"
-#include "nvim/os/rstream_defs.h"
-#include "nvim/os/job_defs.h"
-
-#ifdef INCLUDE_GENERATED_DECLARATIONS
-# include "os/uv_helpers.h.generated.h"
-#endif
-#endif  // NVIM_OS_UV_HELPERS_H
diff --git a/src/nvim/os/wstream.c b/src/nvim/os/wstream.c
deleted file mode 100644
index 7f5191947a..0000000000
--- a/src/nvim/os/wstream.c
+++ /dev/null
@@ -1,243 +0,0 @@
-#include <assert.h>
-#include <stdint.h>
-#include <stdbool.h>
-#include <stdlib.h>
-
-#include <uv.h>
-
-#include "nvim/os/uv_helpers.h"
-#include "nvim/os/wstream.h"
-#include "nvim/os/wstream_defs.h"
-#include "nvim/vim.h"
-#include "nvim/memory.h"
-
-#define DEFAULT_MAXMEM 1024 * 1024 * 10
-
-struct wstream {
-  uv_stream_t *stream;
-  // Memory currently used by pending buffers
-  size_t curmem;
-  // Maximum memory used by this instance
-  size_t maxmem;
-  // Number of pending requests
-  size_t pending_reqs;
-  bool freed, free_handle;
-  // (optional) Write callback and data
-  wstream_cb cb;
-  void *data;
-};
-
-struct wbuffer {
-  size_t size, refcount;
-  char *data;
-  wbuffer_data_finalizer cb;
-};
-
-typedef struct {
-  WStream *wstream;
-  WBuffer *buffer;
-  uv_write_t uv_req;
-} WRequest;
-
-#ifdef INCLUDE_GENERATED_DECLARATIONS
-# include "os/wstream.c.generated.h"
-#endif
-
-/// Creates a new WStream instance. A WStream encapsulates all the boilerplate
-/// necessary for writing to a libuv stream.
-///
-/// @param maxmem Maximum amount memory used by this `WStream` instance. If 0,
-///        a default value of 10mb will be used.
-/// @return The newly-allocated `WStream` instance
-WStream * wstream_new(size_t maxmem)
-{
-  if (!maxmem) {
-    maxmem = DEFAULT_MAXMEM;
-  }
-
-  WStream *rv = xmalloc(sizeof(WStream));
-  rv->maxmem = maxmem;
-  rv->stream = NULL;
-  rv->curmem = 0;
-  rv->pending_reqs = 0;
-  rv->freed = false;
-  rv->free_handle = false;
-  rv->cb = NULL;
-
-  return rv;
-}
-
-/// Frees all memory allocated for a WStream instance
-///
-/// @param wstream The `WStream` instance
-void wstream_free(WStream *wstream) {
-  if (!wstream->pending_reqs) {
-    if (wstream->free_handle) {
-      uv_close((uv_handle_t *)wstream->stream, close_cb);
-    } else {
-      handle_set_wstream((uv_handle_t *)wstream->stream, NULL);
-      xfree(wstream);
-    }
-  } else {
-    wstream->freed = true;
-  }
-}
-
-/// Sets the underlying `uv_stream_t` instance
-///
-/// @param wstream The `WStream` instance
-/// @param stream The new `uv_stream_t` instance
-void wstream_set_stream(WStream *wstream, uv_stream_t *stream)
-{
-  handle_set_wstream((uv_handle_t *)stream, wstream);
-  wstream->stream = stream;
-}
-
-/// Sets the underlying file descriptor that will be written to. Only pipes
-/// are supported for now.
-///
-/// @param wstream The `WStream` instance
-/// @param file The file descriptor
-void wstream_set_file(WStream *wstream, uv_file file)
-{
-  assert(uv_guess_handle(file) == UV_NAMED_PIPE ||
-         uv_guess_handle(file) == UV_TTY);
-  wstream->stream = xmalloc(sizeof(uv_pipe_t));
-  uv_pipe_init(&loop.uv, (uv_pipe_t *)wstream->stream, 0);
-  uv_pipe_open((uv_pipe_t *)wstream->stream, file);
-  wstream->stream->data = NULL;
-  handle_set_wstream((uv_handle_t *)wstream->stream, wstream);
-  wstream->free_handle = true;
-}
-
-/// Sets a callback that will be called on completion of a write request,
-/// indicating failure/success.
-///
-/// This affects all requests currently in-flight as well. Overwrites any
-/// possible earlier callback.
-///
-/// @note This callback will not fire if the write request couldn't even be
-///       queued properly (i.e.: when `wstream_write() returns an error`).
-///
-/// @param wstream The `WStream` instance
-/// @param cb The callback
-/// @param data User-provided data that will be passed to `cb`
-void wstream_set_write_cb(WStream *wstream, wstream_cb cb, void *data)
-  FUNC_ATTR_NONNULL_ARG(1)
-{
-  wstream->cb = cb;
-  wstream->data = data;
-}
-
-/// Queues data for writing to the backing file descriptor of a `WStream`
-/// instance. This will fail if the write would cause the WStream use more
-/// memory than specified by `maxmem`.
-///
-/// @param wstream The `WStream` instance
-/// @param buffer The buffer which contains data to be written
-/// @return false if the write failed
-bool wstream_write(WStream *wstream, WBuffer *buffer)
-{
-  // This should not be called after a wstream was freed
-  assert(!wstream->freed);
-
-  if (wstream->curmem > wstream->maxmem) {
-    goto err;
-  }
-
-  wstream->curmem += buffer->size;
-
-  WRequest *data = xmalloc(sizeof(WRequest));
-  data->wstream = wstream;
-  data->buffer = buffer;
-  data->uv_req.data = data;
-
-  uv_buf_t uvbuf;
-  uvbuf.base = buffer->data;
-  uvbuf.len = buffer->size;
-
-  if (uv_write(&data->uv_req, wstream->stream, &uvbuf, 1, write_cb)) {
-    xfree(data);
-    goto err;
-  }
-
-  wstream->pending_reqs++;
-  return true;
-
-err:
-  wstream_release_wbuffer(buffer);
-  return false;
-}
-
-/// Creates a WBuffer object for holding output data. Instances of this
-/// object can be reused across WStream instances, and the memory is freed
-/// automatically when no longer needed(it tracks the number of references
-/// internally)
-///
-/// @param data Data stored by the WBuffer
-/// @param size The size of the data array
-/// @param refcount The number of references for the WBuffer. This will be used
-///        by WStream instances to decide when a WBuffer should be freed.
-/// @param cb Pointer to function that will be responsible for freeing
-///        the buffer data(passing 'free' will work as expected).
-/// @return The allocated WBuffer instance
-WBuffer *wstream_new_buffer(char *data,
-                            size_t size,
-                            size_t refcount,
-                            wbuffer_data_finalizer cb)
-{
-  WBuffer *rv = xmalloc(sizeof(WBuffer));
-  rv->size = size;
-  rv->refcount = refcount;
-  rv->cb = cb;
-  rv->data = data;
-
-  return rv;
-}
-
-static void write_cb(uv_write_t *req, int status)
-{
-  WRequest *data = req->data;
-
-  data->wstream->curmem -= data->buffer->size;
-
-  wstream_release_wbuffer(data->buffer);
-
-  if (data->wstream->cb) {
-    data->wstream->cb(data->wstream,
-                      data->wstream->data,
-                      status);
-  }
-
-  data->wstream->pending_reqs--;
-
-  if (data->wstream->freed && data->wstream->pending_reqs == 0) {
-    // Last pending write, free the wstream;
-    if (data->wstream->free_handle) {
-      uv_close((uv_handle_t *)data->wstream->stream, close_cb);
-    } else {
-      xfree(data->wstream);
-    }
-  }
-
-  xfree(data);
-}
-
-void wstream_release_wbuffer(WBuffer *buffer)
-{
-  if (!--buffer->refcount) {
-    if (buffer->cb) {
-      buffer->cb(buffer->data);
-    }
-
-    xfree(buffer);
-  }
-}
-
-static void close_cb(uv_handle_t *handle)
-{
-  xfree(handle_get_wstream(handle));
-  xfree(handle->data);
-  xfree(handle);
-}
-
diff --git a/src/nvim/os/wstream.h b/src/nvim/os/wstream.h
deleted file mode 100644
index d0e9bef93a..0000000000
--- a/src/nvim/os/wstream.h
+++ /dev/null
@@ -1,13 +0,0 @@
-#ifndef NVIM_OS_WSTREAM_H
-#define NVIM_OS_WSTREAM_H
-
-#include <stdint.h>
-#include <stdbool.h>
-#include <uv.h>
-
-#include "nvim/os/wstream_defs.h"
-
-#ifdef INCLUDE_GENERATED_DECLARATIONS
-# include "os/wstream.h.generated.h"
-#endif
-#endif  // NVIM_OS_WSTREAM_H
diff --git a/src/nvim/os/wstream_defs.h b/src/nvim/os/wstream_defs.h
deleted file mode 100644
index cfa0bf0b60..0000000000
--- a/src/nvim/os/wstream_defs.h
+++ /dev/null
@@ -1,19 +0,0 @@
-#ifndef NVIM_OS_WSTREAM_DEFS_H
-#define NVIM_OS_WSTREAM_DEFS_H
-
-typedef struct wbuffer WBuffer;
-typedef struct wstream WStream;
-typedef void (*wbuffer_data_finalizer)(void *data);
-
-/// Type of function called when the WStream has information about a write
-/// request.
-///
-/// @param wstream The `WStream` instance
-/// @param data User-defined data
-/// @param status 0 on success, anything else indicates failure
-typedef void (*wstream_cb)(WStream *wstream,
-                           void *data,
-                           int status);
-
-#endif  // NVIM_OS_WSTREAM_DEFS_H
-