Update for Zig 0.17.0#220
Conversation
Signed-off-by: AnErrupTion <anerruption@disroot.org>
Signed-off-by: AnErrupTion <anerruption@disroot.org>
|
I haven't looked closely, but I think Thanks for working on this! This will also need updates to the CI files to use zig 0.17 |
Indeed, this seems to do the trick! |
Signed-off-by: AnErrupTion <anerruption@disroot.org>
Was this merged before fixing the multi threading example? If that is the case, that is fine, but if it's broken I'd like to make sure an issue is opened to track that |
Yes, the multi-threading example is still broken. |
|
ope, my bad, i wasn't following closely enough. in that case i'd like to revert, actually. i'd happily accept a follow-up to this that fixes the multi-threading test! |
I'll re-open it tomorrow, right now it's getting late for me. |
Hi! This PR updates the code base to work on Zig 0.17.0. Specifically, this has been tested on Zig 0.17.0-dev.1282+c0f9b51d8.
All tests run
except the void table ones, becausevoid{}is no longer valid syntax and throws this at compile-time:I'm not sure if this can be fixed or not, so in doubt, I have commented these tests out.EDIT: Fixed!
Also, all examples run and work except the multi-threading one, which compiles but doesn't work most of the time and I'm not sure why. Help appreciated here!