diff options
| author | Andrew Kelley <andrew@ziglang.org> | 2023-01-31 14:02:32 -0700 |
|---|---|---|
| committer | Andrew Kelley <andrew@ziglang.org> | 2023-01-31 15:09:35 -0700 |
| commit | 90e48d4b3469fb4f8dd2f3b52e05453029d45fdc (patch) | |
| tree | c213ee58f8a39ce6275456f5d37a59de4a58cf68 /lib/std/Build/Step.zig | |
| parent | 13a96165405af33fa6ef43a3ce2c1d8aea846287 (diff) | |
| download | zig-90e48d4b3469fb4f8dd2f3b52e05453029d45fdc.tar.gz zig-90e48d4b3469fb4f8dd2f3b52e05453029d45fdc.zip | |
std.Build: avoid use of catch unreachable
Usage of `catch unreachable` in build scripts is completely harmless
because build scripts are always run in Debug mode, however, it sets a
poor example for beginners to learn from.
Diffstat (limited to 'lib/std/Build/Step.zig')
| -rw-r--r-- | lib/std/Build/Step.zig | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/lib/std/Build/Step.zig b/lib/std/Build/Step.zig index aff8a49161..ff0ceb2a51 100644 --- a/lib/std/Build/Step.zig +++ b/lib/std/Build/Step.zig @@ -57,7 +57,7 @@ pub fn init( ) Step { return Step{ .id = id, - .name = allocator.dupe(u8, name) catch unreachable, + .name = allocator.dupe(u8, name) catch @panic("OOM"), .makeFn = makeFn, .dependencies = std.ArrayList(*Step).init(allocator), .loop_flag = false, @@ -77,7 +77,7 @@ pub fn make(self: *Step) !void { } pub fn dependOn(self: *Step, other: *Step) void { - self.dependencies.append(other) catch unreachable; + self.dependencies.append(other) catch @panic("OOM"); } fn makeNoOp(self: *Step) anyerror!void { |