Skip to content

Update to Zig 0.17.0#223

Open
AnErrupTion wants to merge 5 commits into
natecraddock:mainfrom
AnErrupTion:zig-0.17
Open

Update to Zig 0.17.0#223
AnErrupTion wants to merge 5 commits into
natecraddock:mainfrom
AnErrupTion:zig-0.17

Conversation

@AnErrupTion

@AnErrupTion AnErrupTion commented Jul 11, 2026

Copy link
Copy Markdown
Contributor

See #220. The multi-threading example needs to be fixed.

Signed-off-by: AnErrupTion <anerruption@disroot.org>
Signed-off-by: AnErrupTion <anerruption@disroot.org>
Signed-off-by: AnErrupTion <anerruption@disroot.org>
Signed-off-by: AnErrupTion <anerruption@disroot.org>
Signed-off-by: AnErrupTion <anerruption@disroot.org>
@AnErrupTion

Copy link
Copy Markdown
Contributor Author

Well, I'm stupid. I did not see this comment at the beginning of the file:

//! The additional header must be passed to the build using \`-Dlua_user_h=examples/user.h\`

So now it does work, haha. However, it was quite slow due to the allocator, and using the SmpAllocator fixes that issue.

@robbielyman

Copy link
Copy Markdown
Collaborator

It’s probably fine for the examples to be slow; they’re somewhere between tests and model code.

@AnErrupTion

Copy link
Copy Markdown
Contributor Author

It’s probably fine for the examples to be slow; they’re somewhere between tests and model code.

Sure but, as a bonus, the SmpAllocator is guaranteed to be threadsafe, compared to whatever GPA Zig decided to initialise in std.process.Init.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants