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

Add "-Wextra -pedantic -std=c11" to the list of flags #5

Open
ckaran opened this issue Oct 26, 2016 · 0 comments
Open

Add "-Wextra -pedantic -std=c11" to the list of flags #5

ckaran opened this issue Oct 26, 2016 · 0 comments

Comments

@ckaran
Copy link
Contributor

ckaran commented Oct 26, 2016

I've been trying to track down why I keep having issues with code building and passing sometimes, and then failing completely when I haven't modified the code. I finally added -Wextra -pedantic -std=c11 to the list of flags to see what is going on. It turned up a LOT of comparisons between signed and unsigned objects, and it turned up a lot of missing field initializers within the generated code. I suspect that some of the code is accessing uninitialized memory and sometimes works by accident, and sometimes fails, depending on what was in that memory location. Can you please look into this? I don't have time to go through all the warnings right now. I'll push my clang-format branch as it is, which should pass the tests at least most of the time, but cleaning up all the warnings is going to take a LONG time.

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

No branches or pull requests

1 participant