| author |  Alex Rønne Petersen <alex@alexrp.com> 2026-10-13 20:06:22+02:00 |
| committer | Alex Rønne Petersen <alex@alexrp.com> 2026-10-15 22:59:20+02:00 |
| log | 26265e29f4599660a4bfcbc196fc98110bdac6d0 |
| tree | 9f92a464f934fa14f24b7b091abaac2aa0b66996 |
| parent | 6cec052b205b27e89950097e0b5ef1f470dec11a |
| signature | Signed by SSH key SHA256:7B/LJ7bpR1eX8aCXSr4mtd5M45VMPKcx9zY8e95b5QM |
std.c, libzigc: fix ffs() family signatures and implementation bugs
3 files changed, 8 insertions(+), 12 deletions(-)
lib
std
test
lib/c/strings.zig+4-4
| ... | ... | @@ -44,19 +44,19 @@ fn rindex(str: [*:0]const c_char, value: c_int) callconv(.c) ?[*:0]c_char { |
| 44 | 44 | return @constCast(str[std.mem.findScalarLast(u8, std.mem.span(@as([*:0]const u8, @ptrCast(str))), @truncate(@as(c_uint, @bitCast(value)))) orelse return null ..]); |
| 45 | 45 | } |
| 46 | 46 | |
| 47 | fn firstBitSet(comptime T: type, value: T) T { | |
| 48 | return @bitSizeOf(T) - @clz(value); | |
| 47 | fn firstBitSet(comptime T: type, value: T) c_int { | |
| 48 | return if (value == 0) 0 else @as(c_int, @ctz(value)) + 1; | |
| 49 | 49 | } |
| 50 | 50 | |
| 51 | 51 | fn ffs(i: c_int) callconv(.c) c_int { |
| 52 | 52 | return firstBitSet(c_int, i); |
| 53 | 53 | } |
| 54 | 54 | |
| 55 | fn ffsl(i: c_long) callconv(.c) c_long { | |
| 55 | fn ffsl(i: c_long) callconv(.c) c_int { | |
| 56 | 56 | return firstBitSet(c_long, i); |
| 57 | 57 | } |
| 58 | 58 | |
| 59 | fn ffsll(i: c_longlong) callconv(.c) c_longlong { | |
| 59 | fn ffsll(i: c_longlong) callconv(.c) c_int { | |
| 60 | 60 | return firstBitSet(c_longlong, i); |
| 61 | 61 | } |
| 62 | 62 |
lib/std/c.zig+2-2
| ... | ... | @@ -11579,8 +11579,8 @@ pub const wcsdup = switch (builtin.abi) { |
| 11579 | 11579 | }; |
| 11580 | 11580 | |
| 11581 | 11581 | pub extern "c" fn ffs(i: c_int) c_int; |
| 11582 | pub extern "c" fn ffsl(i: c_long) c_long; | |
| 11583 | pub extern "c" fn ffsll(i: c_longlong) c_longlong; | |
| 11582 | pub extern "c" fn ffsl(i: c_long) c_int; | |
| 11583 | pub extern "c" fn ffsll(i: c_longlong) c_int; | |
| 11584 | 11584 | |
| 11585 | 11585 | pub extern "c" fn erand48(xsubi: *[3]c_ushort) f64; |
| 11586 | 11586 | pub extern "c" fn jrand48(xsubi: *[3]c_ushort) c_long; |
test/c/strings.zig+2-6
| ... | ... | @@ -26,9 +26,8 @@ fn testFfs(comptime T: type) !void { |
| 26 | 26 | try testing.expectEqual(0, ffs(0)); |
| 27 | 27 | |
| 28 | 28 | for (0..@bitSizeOf(T)) |i| { |
| 29 | const bit = @as(T, 1) << @intCast(i); | |
| 30 | ||
| 31 | try testing.expectEqual(@as(T, @intCast(i + 1)), ffs(bit)); | |
| 29 | try testing.expectEqual(@as(c_int, @intCast(i + 1)), ffs(@as(T, 1) << @intCast(i))); | |
| 30 | try testing.expectEqual(@as(c_int, @intCast(i + 1)), ffs(@as(T, -1) << @intCast(i))); | |
| 32 | 31 | } |
| 33 | 32 | } |
| 34 | 33 | |
| ... | ... | @@ -41,9 +40,6 @@ test "ffs" { |
| 41 | 40 | if (builtin.target.os.tag == .netbsd) return; // no ffsl/ffsll until 11 |
| 42 | 41 | |
| 43 | 42 | try testFfs(c_long); |
| 44 | ||
| 45 | if (@sizeOf(usize) == 4) return error.SkipZigTest; // TODO | |
| 46 | ||
| 47 | 43 | try testFfs(c_longlong); |
| 48 | 44 | } |
| 49 | 45 |