-
Notifications
You must be signed in to change notification settings - Fork 200
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
Doesn't build on linux #76
Comments
This error is caused by a change in the default behavior of the compiler. If you have an idea how to make it beautiful, we are waiting for the patch =) |
I outlined two solutions. I am not sure though, which one is preferable. |
I think marking these as extern makes the most sense to me. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
You get a lot of linker errors like
The problem is that the
cengine.h
header defines global buffer variables, that are compiled into each file:There are two quick solutions
extern
incengine.h
and define them incengine.c
onceThis is thread-safer as a bonus, but putting 16K on the stack seems somewhat excessive.
The text was updated successfully, but these errors were encountered: