Skip to content
New issue

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

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

Already on GitHub? # to your account

Fix strict-prototype warnings. #970

Merged
merged 1 commit into from
Jan 20, 2024

Conversation

waywardmonkeys
Copy link
Contributor

Need to specify (void) as the empty arg list in C rather than just ().

Need to specify `(void)` as the empty arg list in C rather than
just `()`.
@floooh floooh merged commit b98f8ee into floooh:master Jan 20, 2024
22 of 23 checks passed
@floooh
Copy link
Owner

floooh commented Jan 20, 2024

Thanks!

The Emscripten CI problem is unrelated (looks like the latest Emscripten SDK has some breaking changes in the WebGPU APIs).

@waywardmonkeys waywardmonkeys deleted the fix-strict-prototype-warnings branch January 20, 2024 15:46
@waywardmonkeys
Copy link
Contributor Author

Thanks!

The Emscripten CI problem is unrelated (looks like the latest Emscripten SDK has some breaking changes in the WebGPU APIs).

Maybe I'll take a look at fixing that ... tomorrow perhaps since it is almost 11pm here now on Saturday. :)

@floooh
Copy link
Owner

floooh commented Jan 20, 2024

Don't worry, I'll look into it now. Have a good night :)

@floooh
Copy link
Owner

floooh commented Jan 20, 2024

Ok, I have -Wstrict-prototypes enabled now for the Clang and GCC C builds in the CI tests. Thanks for making me aware of the issue :)

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants