Skip to content
This repository was archived by the owner on Jun 8, 2025. It is now read-only.

Commit 2599309

Browse files
committed
Fix to use testing.allocator for ArenaAllocator
Signed-off-by: Kotaro Inoue <k.musaino@gmail.com>
1 parent b7e4bc0 commit 2599309

File tree

1 file changed

+9
-6
lines changed

1 file changed

+9
-6
lines changed

src/util.zig

Lines changed: 9 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -118,8 +118,9 @@ pub fn mkdirTemp(allocator: Allocator, dir: []const u8) MkdirTempError![]const u
118118
}
119119

120120
test "mkdirTemp" {
121-
// TODO(musaprg): testing.allocator leaks, needs investigation
122-
const allocator = std.heap.ArenaAllocator.init(std.heap.page_allocator).allocator();
121+
var arena = std.heap.ArenaAllocator.init(testing.allocator);
122+
defer arena.deinit();
123+
var allocator = arena.allocator();
123124
const path = try mkdirTemp(allocator, "");
124125
defer {
125126
fs.deleteDirAbsolute(path) catch {};
@@ -158,8 +159,9 @@ pub fn createTempFile(allocator: Allocator, dir: []const u8) CreateTempFileError
158159
}
159160

160161
test "createTempFile" {
161-
// TODO(musaprg): testing.allocator leaks, needs investigation
162-
const allocator = std.heap.ArenaAllocator.init(std.heap.page_allocator).allocator();
162+
var arena = std.heap.ArenaAllocator.init(testing.allocator);
163+
defer arena.deinit();
164+
var allocator = arena.allocator();
163165
const path = try createTempFile(allocator, "");
164166
defer {
165167
fs.deleteFileAbsolute(path) catch {};
@@ -184,8 +186,9 @@ pub fn randomString(allocator: Allocator, random: rand.Random, n: usize) RandomS
184186
}
185187

186188
test "randomString" {
187-
// TODO(musaprg): testing.allocator leaks, needs investigation
188-
const allocator = std.heap.ArenaAllocator.init(std.heap.page_allocator).allocator();
189+
var arena = std.heap.ArenaAllocator.init(testing.allocator);
190+
defer arena.deinit();
191+
var allocator = arena.allocator();
189192
var prng = rand.DefaultPrng.init(0);
190193
const random = prng.random();
191194
_ = try randomString(allocator, random, 10);

0 commit comments

Comments
 (0)