Fix errors in previewer

Don't kill the previewer before waiting. And make sure to deinit the
previewer (was not done before).

Author: natecraddock

src/tui/Previewer.zig

@@ -56,6 +56,7 @@ pub fn startThread(previewer: *Previewer, io: std.Io, loop: *vaxis.Loop(Event))
     previewer.thread = try io.concurrent(threadLoop, .{ previewer, io, loop });
 }
 
+// TODO: can this be cleaned up?
 const ThreadLoopError = error{
     AccessDenied,
     AntivirusInterference,
@@ -154,14 +155,8 @@ pub fn run(gpa: std.mem.Allocator, io: std.Io, argv: []const []const u8) !std.pr
     const stderr_reader = multi_reader.reader(1);
 
     while (multi_reader.fill(64, .none)) |_| {
-        if (stdout_reader.buffered().len > 4 * 4096) {
-            child.kill(io);
-            break;
-        }
-        if (stderr_reader.buffered().len > 4 * 4096) {
-            child.kill(io);
-            break;
-        }
+        if (stdout_reader.buffered().len > 4 * 4096) break;
+        if (stderr_reader.buffered().len > 4 * 4096) break;
     } else |err| switch (err) {
         error.EndOfStream => {},
         else => |e| return e,

src/tui/ui.zig

@@ -130,7 +130,9 @@ pub const State = struct {
         };
     }
 
-    fn deinit(state: *State) void {
+    fn deinit(state: *State, io: std.Io) void {
+        if (state.preview) |*preview| preview.deinit(io);
+
         // We must clear the window because we aren't using the alternate screen
         state.vx.window().clear();
         state.vx.render(state.tty.writer()) catch {};
@@ -144,7 +146,7 @@ pub const State = struct {
         io: std.Io,
         haystacks: [][]const u8,
     ) !?[]const []const u8 {
-        defer state.deinit();
+        defer state.deinit(io);
 
         var filtered = blk: {
             var filtered: ArrayList(Haystack) = try .initCapacity(state.allocator, haystacks.len);