From a1d238e385e44c8e152c4a8aacdb6f518a17054d Mon Sep 17 00:00:00 2001 From: Mitchell Hashimoto Date: Fri, 23 Sep 2022 12:51:50 -0700 Subject: [PATCH] Fix a couple bugs in memory access (tests only) --- src/Command.zig | 4 ++-- src/TempDir.zig | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/src/Command.zig b/src/Command.zig index e798a3f7b..6aa96f393 100644 --- a/src/Command.zig +++ b/src/Command.zig @@ -353,7 +353,7 @@ test "Command: pre exec" { } test "Command: redirect stdout to file" { - const td = try TempDir.init(); + var td = try TempDir.init(); defer td.deinit(); var stdout = try td.dir.createFile("stdout.txt", .{ .read = true }); defer stdout.close(); @@ -378,7 +378,7 @@ test "Command: redirect stdout to file" { } test "Command: custom env vars" { - const td = try TempDir.init(); + var td = try TempDir.init(); defer td.deinit(); var stdout = try td.dir.createFile("stdout.txt", .{ .read = true }); defer stdout.close(); diff --git a/src/TempDir.zig b/src/TempDir.zig index d1850f128..d33f6d58e 100644 --- a/src/TempDir.zig +++ b/src/TempDir.zig @@ -52,13 +52,13 @@ pub fn init() !TempDir { /// Name returns the name of the directory. This is just the basename /// and is not the full absolute path. -pub fn name(self: TempDir) []const u8 { +pub fn name(self: *TempDir) []const u8 { return std.mem.sliceTo(&self.name_buf, 0); } /// Finish with the temporary directory. This deletes all contents in the /// directory. This is safe to call multiple times. -pub fn deinit(self: TempDir) void { +pub fn deinit(self: *TempDir) void { self.parent.deleteTree(self.name()) catch |err| log.err("error deleting temp dir err={}", .{err}); }