diff options
| author | Matthew Lugg <mlugg@mlugg.co.uk> | 2025-12-16 12:05:40 +0000 |
|---|---|---|
| committer | Matthew Lugg <mlugg@mlugg.co.uk> | 2025-12-22 11:54:57 +0000 |
| commit | 49f17237b0332680cebf3e4d83dedc3db46356e9 (patch) | |
| tree | 5e4611540299e30e6a7537291fec8d7389215aa8 /src/Sema.zig | |
| parent | aa0249d74e573742db3567f589fc6e4a00e1fff8 (diff) | |
| download | zig-49f17237b0332680cebf3e4d83dedc3db46356e9.tar.gz zig-49f17237b0332680cebf3e4d83dedc3db46356e9.zip | |
compiler,std: remove a handful of redundant discards
I'm embarrassed to say I didn't realise until a few days ago that you
could omit the `_` capture for `if`/`while` on `E!void`...
Diffstat (limited to 'src/Sema.zig')
| -rw-r--r-- | src/Sema.zig | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/src/Sema.zig b/src/Sema.zig index baeb9a62de..ffb0b20e49 100644 --- a/src/Sema.zig +++ b/src/Sema.zig @@ -1061,7 +1061,7 @@ fn analyzeInlineBody( /// The index which a break instruction can target to break from this body. break_target: Zir.Inst.Index, ) CompileError!?Air.Inst.Ref { - if (sema.analyzeBodyInner(block, body)) |_| { + if (sema.analyzeBodyInner(block, body)) { return null; } else |err| switch (err) { error.ComptimeBreak => {}, @@ -1808,7 +1808,7 @@ fn analyzeBodyInner( child_block.instructions = block.instructions; defer block.instructions = child_block.instructions; - const break_result: ?BreakResult = if (sema.analyzeBodyInner(&child_block, inline_body)) |_| r: { + const break_result: ?BreakResult = if (sema.analyzeBodyInner(&child_block, inline_body)) r: { break :r null; } else |err| switch (err) { error.ComptimeBreak => brk_res: { @@ -1956,7 +1956,7 @@ fn analyzeBodyInner( .@"defer" => blk: { const inst_data = sema.code.instructions.items(.data)[@intFromEnum(inst)].@"defer"; const defer_body = sema.code.bodySlice(inst_data.index, inst_data.len); - if (sema.analyzeBodyInner(block, defer_body)) |_| { + if (sema.analyzeBodyInner(block, defer_body)) { // The defer terminated noreturn - no more analysis needed. break; } else |err| switch (err) { @@ -1975,7 +1975,7 @@ fn analyzeBodyInner( const err_code = try sema.resolveInst(inst_data.err_code); try map.ensureSpaceForInstructions(sema.gpa, defer_body); map.putAssumeCapacity(extra.remapped_err_code, err_code); - if (sema.analyzeBodyInner(block, defer_body)) |_| { + if (sema.analyzeBodyInner(block, defer_body)) { // The defer terminated noreturn - no more analysis needed. break; } else |err| switch (err) { @@ -5959,7 +5959,7 @@ fn resolveBlockBody( assert(sema.air_instructions.items(.tag)[@intFromEnum(merges.block_inst)] == .block); var need_debug_scope = false; child_block.need_debug_scope = &need_debug_scope; - if (sema.analyzeBodyInner(child_block, body)) |_| { + if (sema.analyzeBodyInner(child_block, body)) { return sema.resolveAnalyzedBlock(parent_block, src, child_block, merges, need_debug_scope); } else |err| switch (err) { error.ComptimeBreak => { |