Update zig examples and template to latest Zig #2298
Conversation
| lib.export_symbol_names = &[_][]const u8{ "TIC", "OVR", "BDR", "SCR", "BOOT" }; | ||
| lib.install(); | ||
|
|
||
| lib.export_symbol_names = &[_][]const u8{ "TIC", "OVR", "BDR", "BOOT" }; |
There was a problem hiding this comment.
Has SCR been entirely removed now or just deprecated? If the latter we many want to leave it here for backwards compatibility...
There was a problem hiding this comment.
I don't think someone used it, first of all, it was initially called SCN, and second, most langs don't support SCN (only Lua and JS). We can remove it without any doubts.
| export fn BDR() void {} | ||
|
|
||
| export fn OVR() void {} |
There was a problem hiding this comment.
Are these necessary now to compile?
There was a problem hiding this comment.
yep, if these are not present then the Zig compiler will complain that it couldn't find the symbols we're telling it to export.
| lib.export_symbol_names = &[_][]const u8{ "TIC", "OVR", "BDR", "SCR", "BOOT" }; | ||
| lib.install(); | ||
|
|
||
| lib.export_symbol_names = &[_][]const u8{ "TIC", "OVR", "BDR", "BOOT" }; |
There was a problem hiding this comment.
I don't think someone used it, first of all, it was initially called SCN, and second, most langs don't support SCN (only Lua and JS). We can remove it without any doubts.
The previous Zig
buildandtic80.zigfiles built fine with Zig 0.10, but no longer build with the latest stable (0.11) nor the current development version (0.12).This PR updates the Zig examples so that they build with both 0.11 and 0.12 😄
I've also formatted the files in question with Zig's language server formatter.