Does not compile on latest Zig

joachimschmidt557 / linenoize

A port of linenoise to zig

MIT License

55 stars 9 forks source link

Does not compile on latest Zig #20

Closed notramo closed 10 months ago

notramo commented 11 months ago

sts.mem.copy should be replaced with @memcpy

Phytolizer commented 10 months ago

FWIW the semantics differ a fair amount: with @memcpy, the slices need to be the same length and also not overlap. The direct replacement is std.mem.copyForwards.

joachimschmidt557 commented 10 months ago

Thanks for the info, I've solved the issue with 60da07356154218ea4d427c258aa0fab093eb4f3.

Phytolizer commented 10 months ago

The fix is incomplete, there are 2 more usages in term.zig: https://github.com/joachimschmidt557/linenoize/blob/master/src/term.zig#L167-L170

joachimschmidt557 commented 10 months ago

Thanks, fixed with a69e3764e32523c28717113da309bbee5dab66c4

Phytolizer commented 10 months ago

Thanks!