Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix compiler errors - StackTrace and unused parameter. #10

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

lukekras
Copy link

@lukekras lukekras commented Dec 6, 2021

I can see a few PRs with the same fix. @andrewrk probably hasn't looked at this repo in a while.

Fixed minor compiling issues, due to changes in Zig compiler. builtin.StackTrace is now under std. Also Zig complains that error_return_trace is not being used. I toyed with doing some comptime checks of where StackTrace was located, but thought that the simpler changes were probably better. Also people are most probably using new versions of the Zig compiler anyway.

Tested on:
MacOS Big Sur 11.6.1 Intel

% qemu-system-i386 -version
QEMU emulator version 6.1.0
% zig version
0.9.0-dev.1857+14b532ec8

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.

1 participant