diff options
| author | Isaac Freund <mail@isaacfreund.com> | 2023-01-22 17:34:38 +0100 |
|---|---|---|
| committer | Isaac Freund <mail@isaacfreund.com> | 2023-01-23 12:19:53 +0100 |
| commit | faf2fd18d33de246c5bb03905f25290108e1dd30 (patch) | |
| tree | 76549a3a52ed3d3dac3fd8f62d92a1e90e95c721 /lib/std/debug.zig | |
| parent | c0284e242f7d78955204dc8a627fecd45aa5e521 (diff) | |
| download | zig-faf2fd18d33de246c5bb03905f25290108e1dd30.tar.gz zig-faf2fd18d33de246c5bb03905f25290108e1dd30.zip | |
std: eliminate pointless meta.assumeSentinel() usage
This fixes a bug in std.net caused during the introduction of
meta.assumeSentinel due to the unfortunate semantics of mem.span()
This leaves only 3 remaining uses of meta.assumeSentinel() in the
standard library, each of which could be a simple @ptrCast([*:0]T, foo)
instead. I think this function should likely be removed.
Diffstat (limited to 'lib/std/debug.zig')
| -rw-r--r-- | lib/std/debug.zig | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/std/debug.zig b/lib/std/debug.zig index 99ddb266b6..3708c4fe81 100644 --- a/lib/std/debug.zig +++ b/lib/std/debug.zig @@ -978,7 +978,7 @@ pub fn readElfDebugInfo(allocator: mem.Allocator, elf_file: File) !ModuleDebugIn for (shdrs) |*shdr| { if (shdr.sh_type == elf.SHT_NULL) continue; - const name = std.mem.span(std.meta.assumeSentinel(header_strings[shdr.sh_name..].ptr, 0)); + const name = mem.sliceTo(header_strings[shdr.sh_name..], 0); if (mem.eql(u8, name, ".debug_info")) { opt_debug_info = try chopSlice(mapped_mem, shdr.sh_offset, shdr.sh_size); } else if (mem.eql(u8, name, ".debug_abbrev")) { |